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

Broken a big file into smaller files. Added .test suffix for test files. #208

Merged
merged 4 commits into from
Oct 12, 2022

Conversation

0xorial
Copy link
Collaborator

@0xorial 0xorial commented Oct 11, 2022

No description provided.

@0xorial 0xorial changed the title Broken a big file into smaller files. Broken a big file into smaller files. Added .test suffix for test files. Oct 11, 2022
@0xorial 0xorial linked an issue Oct 11, 2022 that may be closed by this pull request
@rhys-vdw
Copy link
Owner

Haha, I hope you automated this process. 😆

All tests appear to be passing so LGTM! Does this not close #192?

@rhys-vdw rhys-vdw merged commit a60f88b into rhys-vdw:master Oct 12, 2022
@0xorial
Copy link
Collaborator Author

0xorial commented Oct 14, 2022

hehe, yeah, you can see that in the intermediate commit.
I guess we indeed can close it.
meanwhile I kinda go hooked on TAP and looking around if we can make it great :)

@rhys-vdw
Copy link
Owner

re tape-testing/tape#582 (comment)

I'm on windows and can run the test suite just fine. Although I didn't test it after your refactor (just trusted CI)

@rhys-vdw
Copy link
Owner

meanwhile I kinda go hooked on TAP

It's awesome isn't it? I love how simple it is.

@0xorial
Copy link
Collaborator Author

0xorial commented Oct 17, 2022

I'm on windows and can run the test suite just fine. Although I didn't test it after your refactor (just trusted CI)

tape itself runs on windows, I just have problem with running the tests of the tape itself...

It's awesome isn't it? I love how simple it is.

There is a certain OCD joy in having a spec dictating how test runner should interact with the host (test runners/frameworks seem practically identical features reporting-wise, even if I take .net frameworks into account, so it would be so much cooler if IDEs could just support TAP-compatible runner (although this would probably require to include CLI parameters specs in the TAP) :)

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.

2 participants