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

migrate from Test::More to Test2::V0 #72

Closed
wants to merge 2 commits into from
Closed

Conversation

yanick
Copy link
Collaborator

@yanick yanick commented Oct 4, 2019

As My Life with the Thrill Test Kult would say

I am the new way to Test2::V0
I am the way of the future

(seriously, this PR is mostly for giggles, and because if we don't promote and lead the way to Perl evolution, who else will?)

@coveralls
Copy link

coveralls commented Oct 4, 2019

Coverage Status

Coverage remained the same at 76.382% when pulling f9ce705 on yanick:test2 into 4f1764c on genehack:master.

@@ -45,7 +45,7 @@ $ENV{GITGOT_FAKE_GIT_WRAPPER} = 1;
my $entry = LoadFile( $config );
is( $entry->[0]{name} , 'fake-git-repo' , 'expected name' );
is( $entry->[0]{type} , 'git' , 'expected type' );
is( $entry->[0]{path} , path( "$dir/fake-git-repo" ) , 'expected path' );
is( $entry->[0]{path} , ''.path( "$dir/fake-git-repo" ) , 'expected path' );
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is the "append to an empty string" bit needed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

path is a Path::tiny object, and is won't auto-overload it into its string representation iirc.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ugh.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is ->stringify if you want a more couth version

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yay.

@genehack
Copy link
Owner

So, I don't see the point of converting to Test2::V0 unless we can completely convert to it, and I don't see any equivalent of new_ok() that would allow the replacement of the Test::More usage in t/lib/Test/App/GitGot/Repo.pm and t/lib/Test/App/GitGot/Repo/Git.pm.

What am I missing?

@yanick
Copy link
Collaborator Author

yanick commented Apr 26, 2020

What am I missing?

6 lines.

New commit pushed.

@genehack
Copy link
Owner

So, yeah, if this is going to end up adding additional testing code that I have to maintain going forward, I'm not interested.

It's one thing if it's just swapping one dep for another and changing some function call signatures; maintaining extra code that is maintained for me today is not an improvement in my book.

@genehack genehack closed this Apr 28, 2020
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

3 participants