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
pythonPackages.robotframework-selenium2library: 1.6.0 -> 3.0.0 #51731
pythonPackages.robotframework-selenium2library: 1.6.0 -> 3.0.0 #51731
Conversation
fyi, since is a major version upgrade, I've checked with nix-review locally and it didn't throw any error |
pkgs/development/python-modules/robotframework-selenium2library/default.nix
Outdated
Show resolved
Hide resolved
pkgs/development/python-modules/robotframework-seleniumlibrary/default.nix
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should a different sourcecode location be fetched, if it provides tests?
pkgs/development/python-modules/robotframework-selenium2library/default.nix
Outdated
Show resolved
Hide resolved
pkgs/development/python-modules/robotframework-seleniumlibrary/default.nix
Show resolved
Hide resolved
b5d2672
to
ffa3e27
Compare
@worldofpeace, @das-g thanks for your review. I've updated the PR with your suggestions. |
pkgs/development/python-modules/robotframework-seleniumlibrary/default.nix
Outdated
Show resolved
Hide resolved
ffa3e27
to
b7201f6
Compare
}; | ||
|
||
propagatedBuildInputs = [ robotframework selenium ]; | ||
checkInputs = [ mockito robotstatuschecker approvaltests ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would testing robotframework-seleniumlibrary with approvaltests
start an interactive diff/merge tool (whether graphical or CLI), when the respective tests fail?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it will depend whether if the approval reporter is available (e.g. non-sandboxed builds), otherwise it will just throw an ApprovalException
exception
''; | ||
|
||
meta = with stdenv.lib; { | ||
description = "Spying framework"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While the README of the mockito-python indeed calls it a "spying framework", I think that label is somewhat confusing out of context. It's only for "spying" on the system-under-test, specifically how that uses the provided mocks, partial mocks and stubs. It's not for spying on, say, people or organizations.
Thus I'd go with how the Java library of the same name calls itself: A "mocking framework".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm personally inclined to use the description used by the project itself, but I can change it if you prefer it
|
||
propagatedBuildInputs = [ robotframework selenium docutils decorator ]; | ||
propagatedBuildInputs = [ robotframework-seleniumlibrary ]; | ||
|
||
meta = with stdenv.lib; { | ||
description = "Web testing library for Robot Framework"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the description indicate, that this is for helping transitioning and that the new name is robotframework-seleniumlibrary
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure, I think adding a to the project homepage should be enough; but if you prefer it I can add it to longDescription
|
||
meta = with stdenv.lib; { | ||
description = "Web testing library for Robot Framework"; | ||
homepage = http://robotframework.org/; | ||
homepage = https://github.com/robotframework/Selenium2Library; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
b7201f6
to
93805bb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new packages are missing meta.maintainers
.
93805bb
to
cf46e28
Compare
Fixed, I added myself as maintainer |
@GrahamcOfBorg build python2.pkgs.approvaltests python3.pkgs.approvaltests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ran nix-review
with success
I was interested in Darwin specifically and as everything seems to work there, I think this is good to go! |
@marsam Many thanks again 🎆 |
Motivation for this change
Fixes #51727
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)