Skip to content

Pass problematic tests #82

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

Closed
wants to merge 239 commits into from
Closed

Conversation

khatchad
Copy link
Collaborator

@khatchad khatchad commented Jul 2, 2023

That way, when we fix the bug, the tests will fail, in which case we will know that our fix is working. Also, we won't forget to uncomment the problematic tests.

We're getting it from the parent pom.
It would seem that the test files are in the normal source directories.
Not sure if it's completely necessary.
The token isn't working.
I think it is included in the deployment.
Different projects have different build and resource directories.
khatchad and others added 24 commits June 27, 2023 16:57
Adding log for the dataflow sources
…_test

Add test having a decorated function with a tensor parameter.
* Adding log for analysis results

* Shorten.

---------

Co-authored-by: Raffi Khatchadourian <raffi.khatchadourian@hunter.cuny.edu>
Otherwise, we get a warning that it is empty.
Fix extra function caused by additional summary entries for tensor generators
That way, when we fix the bug, the tests will fail, in which case we will know that our fix is working. Also, we won't forget to uncomment the problematic tests.
@khatchad khatchad closed this Jul 2, 2023
@khatchad
Copy link
Collaborator Author

khatchad commented Jul 2, 2023

Superseded by #83.

@khatchad khatchad added the wontfix This will not be worked on label Jul 2, 2023
@khatchad khatchad deleted the pass_problematic_tests branch July 2, 2023 18:35
khatchad added a commit that referenced this pull request Feb 5, 2024
* Add test cases for #140.

* Fix #140.
khatchad added a commit that referenced this pull request Jul 8, 2025
* Add test cases for #140.

* Fix #140.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants