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

Remove MEP tests results. Move tests from Hypar.Tests namespace. #1037

Merged
merged 5 commits into from
Oct 10, 2023

Conversation

DmytroMuravskyi
Copy link
Contributor

@DmytroMuravskyi DmytroMuravskyi commented Oct 6, 2023

BACKGROUND:

  • When working with Elements after MEP was moved inside, I noticed that after each run of unit tests I have a lot of modified files, detected by git from test results folder.
  • I also noticed that several files use Hypar.Test namespace while all other starts with Elements namespace.

DESCRIPTION:

  • Removed MEP test results from repo by deleting existing results and renaming result folder to "RestResults" that is ignored due to a rule inside gitignore.
  • Used Elements.Geometry.Tests instead oh Hypar.Tests in 6 files where it was used, so everything looks standardized.

TESTING:

  • No new tests added.

REQUIRED:

  • All changes are up to date in CHANGELOG.md. Nothing is added to CHANGELOG since changes don't affect any logic.

This change is Reviewable

@wynged wynged self-requested a review October 10, 2023 20:35
@wynged wynged merged commit 10d8ca2 into hypar-io:master Oct 10, 2023
1 check passed
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