Skip to content

Conversation

@jglick
Copy link
Member

@jglick jglick commented Nov 13, 2020

In jenkinsci/bom#359 the test from #23 fails when jth.jenkins-war.path is set to the megawar. Seems that SignupPage.submit quietly returns even when the signup in fact fails. Trying the page interactively showed that it would refuse to proceed unless you entered an email address. Making this change makes the test pass when jth.jenkins-war.path is set. The test utility predates the new signup page GUI but it is not clear that is relevant; I have not figured out what about the megawar (bundled JavaScript?) would change behavior here.

jglick added a commit to jglick/bom that referenced this pull request Nov 13, 2020
@daniel-beck
Copy link
Member

I have not figured out what about the megawar (bundled JavaScript?)

Presence of mailer plugin?

@jglick
Copy link
Member Author

jglick commented Nov 13, 2020

Oh, perhaps.

@jglick
Copy link
Member Author

jglick commented Nov 13, 2020

Hmm, apparently needs to catch ElementNotFoundException in case mailer is absent. On mobile, can try that on Monday.

@jglick
Copy link
Member Author

jglick commented Nov 16, 2020

@daniel-beck
Copy link
Member

Now it failed in

, rerunning.

@jglick
Copy link
Member Author

jglick commented Nov 16, 2020

Hmm, passed locally, not sure what that is about.

@daniel-beck daniel-beck self-requested a review January 19, 2021 12:13
@daniel-beck daniel-beck merged commit 0f3711d into jenkinsci:master Jan 20, 2021
@jglick jglick deleted the PermissionAdderTest branch January 20, 2021 22:57
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