Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

Default tree refactoring to use file manager of package stream #568

Merged

Conversation

brunats
Copy link
Contributor

@brunats brunats commented Sep 30, 2020

- What I did

In this PR we exchange the use of io/ioutil and /fileutil to FileReadExister in all functions of default_tree.go file.

- How to verify it

Check the file default tree.go to see the difference and in the tests of the related file check the use of interface.

- Description for the changelog

Default tree refactoring to use file manager of package stream

Signed-off-by: Bruna Tavares <[email protected]>
Signed-off-by: Bruna Tavares <[email protected]>
Signed-off-by: Bruna Tavares <[email protected]>
Signed-off-by: Bruna Tavares <[email protected]>
Signed-off-by: Bruna Tavares <[email protected]>
…ent/added-tests-to-default-tree

Signed-off-by: Bruna Tavares <[email protected]>
Signed-off-by: Bruna Tavares <[email protected]>
…ent/default-tree-refactoring

Signed-off-by: Bruna Tavares <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Sep 30, 2020

Codecov Report

Merging #568 into master will increase coverage by 0.31%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #568      +/-   ##
==========================================
+ Coverage   80.06%   80.38%   +0.31%     
==========================================
  Files          98       98              
  Lines        3196     3196              
==========================================
+ Hits         2559     2569      +10     
+ Misses        448      443       -5     
+ Partials      189      184       -5     
Impacted Files Coverage Δ
pkg/formula/tree/default_tree.go 100.00% <100.00%> (+18.18%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b8c379d...0e71ad0. Read the comment docs.

@brunasilvazup brunasilvazup changed the title Improvement/default tree refactoring Default tree refactoring to use file manager of package stream Oct 1, 2020
@brunasilvazup brunasilvazup added 🔨 improvement Improvement in features and removed 🚧 WIP Work in Progress labels Oct 1, 2020
@brunasilvazup brunasilvazup added the ✔️ ready-for-review ready for review label Oct 1, 2020
Signed-off-by: Bruna Tavares <[email protected]>
@victor-schumacher
Copy link
Contributor

🥇

@lucasdittrichzup
Copy link
Contributor

🚀

@brunasilvazup
Copy link
Contributor

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Oct 2, 2020

👌 Merged branch improvement/default-tree-refactoring into qa

@brunasilvazup brunasilvazup merged commit 555ee9d into ZupIT:master Oct 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🔨 improvement Improvement in features ✔️ ready-for-review ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants