Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor tests to use golden test files #200

Merged
merged 1 commit into from
May 13, 2020
Merged

Refactor tests to use golden test files #200

merged 1 commit into from
May 13, 2020

Conversation

dsnet
Copy link
Collaborator

@dsnet dsnet commented May 13, 2020

Refactor the unit tests to read the diffs from a file
rather than being stored as a Go string literal.
The advantage of using a golden file is to ease
updating the diffs whenever the reporter output is changed.

@dsnet
Copy link
Collaborator Author

dsnet commented May 13, 2020

\cc cybrcodr for review. I don't know why I can't assign you as reviewer anymore.

Copy link

@cybrcodr cybrcodr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if it's because I'm no longer part of the "google" github org. I should make sure to request that back.

Copy link

@cybrcodr cybrcodr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I got back now.

Refactor the unit tests to read the diffs from a file
rather than being stored as a Go string literal.
The advantage of using a golden file is to ease
updating the diffs whenever the reporter output is changed.
@dsnet
Copy link
Collaborator Author

dsnet commented May 13, 2020

Ok, I got back now.

That was fast.

@dsnet dsnet merged commit 0c08307 into master May 13, 2020
@dsnet dsnet deleted the golden-test branch May 13, 2020 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants