Skip to content

Adding a New User to Market Application#87

Merged
arcuri82 merged 2 commits intodevelopfrom
market-add-user
Oct 31, 2024
Merged

Adding a New User to Market Application#87
arcuri82 merged 2 commits intodevelopfrom
market-add-user

Conversation

@onurd86
Copy link
Copy Markdown
Contributor

@onurd86 onurd86 commented Oct 31, 2024

Add a new user to market app. It utilizes BCrypt Encoder for storing passwords. So, it was just adding a new entry to the database script.

…passwords. So, it was just adding a new entry to the database script.
@onurd86 onurd86 requested a review from arcuri82 October 31, 2024 01:19
@@ -138,7 +138,8 @@ public SutInfoDto.OutputFormat getPreferredOutputFormat() {
public List<AuthenticationDto> getInfoForAuthentication() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

need to update ExternalEvoMasterController as well

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@onurd86 btw, I assume you then run EvoMaster on this update settings, and verified there was no message saying 401 with this new user, isn't it?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hi Andrea,

Thanks for mentioning that. I tried to run EvoMaster again after adding the new user. I do not get any 401 after adding the new user. Adding a new user to market was very easy.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have updated External Controller as well.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This was one of the successful tests generated by EvoMaster after adding user2:

@Test @Timeout(60)
   public void test_1() throws Exception {
       
       given().accept("*/*")
               .header("Authorization", "Basic dXNlcjJAeWFuZGV4LnJ1Onl1cmlkb2xnb3J1a2k=") // user2
               .header("x-EMextraHeader123", "")
               .get(baseUrlOfSut + "/customer/orders?name=ATX")
               .then()
               .statusCode(200)
               .assertThat()
               .contentType("application/json")
               .body("size()", equalTo(0));
   }

@onurd86 onurd86 requested a review from arcuri82 October 31, 2024 08:58
@arcuri82 arcuri82 merged commit 2d288c2 into develop Oct 31, 2024
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