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

fix(test): typedoc signature test issues #2280

Merged
merged 4 commits into from
Aug 3, 2023
Merged

Conversation

ST-DDT
Copy link
Member

@ST-DDT ST-DDT commented Jul 29, 2023

This PRs tries to fix the frequent issues with macOS and our signature tests.
Unfortunately, I don't have a way to test it.

I explicitly ignore the temp folder in the tsconfig to avoid issues with temp files that are detected by either concurrently running tests with typedoc or files being removed async although the should be removed sync and thus being still there when typedoc runs again on the project.

@ST-DDT ST-DDT added p: 1-normal Nothing urgent c: test labels Jul 29, 2023
@ST-DDT ST-DDT requested review from a team July 29, 2023 10:20
@ST-DDT ST-DDT self-assigned this Jul 29, 2023
@codecov
Copy link

codecov bot commented Jul 29, 2023

Codecov Report

Merging #2280 (252acab) into next (87eca26) will increase coverage by 0.00%.
The diff coverage is n/a.

❗ Current head 252acab differs from pull request most recent head 8c7e781. Consider uploading reports for the commit 8c7e781 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #2280   +/-   ##
=======================================
  Coverage   99.59%   99.60%           
=======================================
  Files        2641     2641           
  Lines      244831   244831           
  Branches     1082     1082           
=======================================
+ Hits       243846   243855    +9     
+ Misses        958      949    -9     
  Partials       27       27           

see 2 files with indirect coverage changes

@ST-DDT ST-DDT force-pushed the fix/typedoc-test-issues branch from b82fc49 to c414854 Compare July 29, 2023 11:11
The folder should be deleted immediately after the test runs, but on
macOS it exists long enough for typedoc to pick it up and the complain
about the files being deleted.
@ST-DDT ST-DDT force-pushed the fix/typedoc-test-issues branch from c414854 to 3ba2606 Compare July 29, 2023 11:12
@ST-DDT ST-DDT added the c: bug Something isn't working label Jul 29, 2023
@ST-DDT ST-DDT requested a review from xDivisionByZerox July 31, 2023 07:44
@ST-DDT ST-DDT enabled auto-merge (squash) August 3, 2023 19:17
@ST-DDT ST-DDT merged commit 02fc7ca into next Aug 3, 2023
@ST-DDT ST-DDT deleted the fix/typedoc-test-issues branch August 3, 2023 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: bug Something isn't working c: test p: 1-normal Nothing urgent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants