Project

General

Profile

Bug #78951

Fix regression that prevented user creation without setting email address

Added by Eric Turgeon almost 3 years ago. Updated almost 3 years ago.

Status:
Done
Priority:
No priority
Assignee:
William Grzybowski
Category:
Middleware
Target version:
Seen in:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

https://builds.ixsystems.com/jenkins/job/FreeNAS%20-%2011.2%20-%20Internal%20(QA%20Tests)/job/11.2-stable/lastCompletedBuild/testReport/api1/user/Run_Tests___API_1_0___test_02_Creating_user_testuser/

def test_02_Creating_user_testuser():
        payload = {"bsdusr_username": "testuser",
                   "bsdusr_creategroup": "true",
                   "bsdusr_full_name": "Test User",
                   "bsdusr_password": "test",
                   "bsdusr_uid": 1111,
                   "bsdusr_home": f"/mnt/{pool_name}/testuser",
                   "bsdusr_mode": "755",
                   "bsdusr_shell": "/bin/csh"}
        results = POST("/account/users/", payload)
>       assert results.status_code == 201, results.text
E       AssertionError: {"__all__": ["Should be valid E-Mail address"]}
E       assert 409 == 201
E        +  where 409 = <Response [409]>.status_code

https://builds.ixsystems.com/jenkins/job/FreeNAS%20-%2011.2%20-%20Internal%20(QA%20Tests)/job/11.2-stable/lastCompletedBuild/testReport/api1/ssh/Run_Tests___API_1_0___test_06_Add_ssh_ky_to_root/

def test_06_Add_ssh_ky_to_root():
        payload = {"bsdusr_sshpubkey": sshKey}
        results = PUT("/account/users/1/", payload)
>       assert results.status_code == 200, results.text
E       AssertionError: {"__all__": ["Should be valid E-Mail address"]}
E       assert 409 == 200
E        +  where 409 = <Response [409]>.status_code

History

#1 Updated by Dru Lavigne almost 3 years ago

  • Assignee changed from Release Council to William Grzybowski

#2 Updated by Dru Lavigne almost 3 years ago

  • Target version changed from Backlog to 11.2-U3

#3 Updated by Bug Clerk almost 3 years ago

  • Status changed from Unscreened to In Progress

#4 Updated by Bug Clerk almost 3 years ago

  • Status changed from In Progress to Ready for Testing

#5 Updated by Dru Lavigne almost 3 years ago

  • Subject changed from API v1 POST /account/users/ and PUT /account/users/1/ Response 409 error msg "Should be valid E-Mail address" to Fix regression that prevented user creation without setting email address
  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#7 Updated by Dru Lavigne almost 3 years ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF