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.

Sign up for GitHub

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

Jump to bottom

Resolving Unit Tests Issue #331

Draft
wants to merge 13 commits into
base: stg
Choose a base branch
from
Draft

Resolving Unit Tests Issue #331

wants to merge 13 commits into from

Conversation

J3suis
Copy link
Contributor

@J3suis J3suis commented Jul 16, 2021 •
edited

🆗 Debug Tests
✅ Optimization
❌ New Tests

SUMMARY

  • Optimization of File csv_test to implement Reflection
  • Fix Problem with compilation Of Test

@J3suis J3suis changed the title Unit Test: PR Tempory Resolving Unit Tests Issue Jul 16, 2021
@GabrielRavier
Copy link
Contributor

GabrielRavier commented Oct 8, 2021 •
edited

Updated it to make basically all the tests work properly, although there is still one in ES that I can't seem to comprehend (help would be appreciated here tbh). Should otherwise be ready for review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GabrielRavier GabrielRavier GabrielRavier requested changes

@EthanHusser EthanHusser Awaiting requested review from EthanHusser

@simonmeyerrr simonmeyerrr Awaiting requested review from simonmeyerrr

@cruffinoni cruffinoni Awaiting requested review from cruffinoni

Assignees
No one assigned
Labels
work in progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@J3suis @GabrielRavier