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

Removing file util package with test refactor #830

Merged
merged 4 commits into from
Jan 22, 2021

Conversation

henriquemoraeszup
Copy link
Contributor

Signed-off-by: Henrique Moraes henrique.moraes@zup.com.br

Description

Removing the file util package. Since it was used on the workspace package, I took the liberty of also refactoring workspace tests.

How to verify it

The project should pass on the ci pipeline and not break on your machine

Changelog

Removed deprecated package fileutil

Signed-off-by: Henrique Moraes <henrique.moraes@zup.com.br>
Signed-off-by: Henrique Moraes <henrique.moraes@zup.com.br>
@brunasilvazup brunasilvazup added waiting reply Waiting for an answer to a comment and removed ✔️ ready-for-review ready for review labels Jan 19, 2021
Signed-off-by: Henrique Moraes <henrique.moraes@zup.com.br>
@codecov-io
Copy link

Codecov Report

Merging #830 (905dcf1) into master (37ab4d1) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #830      +/-   ##
==========================================
- Coverage   84.34%   84.31%   -0.03%     
==========================================
  Files         111      111              
  Lines        3857     3852       -5     
==========================================
- Hits         3253     3248       -5     
  Misses        432      432              
  Partials      172      172              
Impacted Files Coverage Δ
pkg/commands/builder.go 90.30% <100.00%> (ø)
pkg/formula/workspace/workspace.go 78.68% <100.00%> (-1.62%) ⬇️

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 37ab4d1...905dcf1. Read the comment docs.

@henriquemoraeszup henriquemoraeszup added ✔️ ready-for-review ready for review and removed waiting reply Waiting for an answer to a comment labels Jan 22, 2021
Signed-off-by: Henrique Moraes <henrique.moraes@zup.com.br>
@lucasdittrichzup lucasdittrichzup merged commit e271fdb into ZupIT:master Jan 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove fileutil package Tests with Testify in Pkg/Formula/Workspace
4 participants