nickva commented on issue #1111: Fix couch_peruser_test
URL: https://github.com/apache/couchdb/pull/1111#issuecomment-357720916
 
 
   Nice find!
   
   Actually this fix seems to discover and runs more tests:
   
   master:
   ```
   ======================== EUnit ========================
   module 'couch_peruser_app'
   module 'couch_peruser_sup'
   module 'couch_peruser_test'
     couch_peruser test
   Application crypto was left running!
       couch_peruser_test:165: should_create_anon_user_db_with_default...[0.002 
s] ok
       couch_peruser_test:166: should_create_anon_user_db_with_default...ok
       couch_peruser_test:155: should_create_user_db_with_default...[0.002 s] ok
       couch_peruser_test:156: should_create_user_db_with_default...ok
       couch_peruser_test:178: should_create_user_db_with_q4...[0.002 s] ok
       couch_peruser_test:179: should_create_user_db_with_q4...ok
       couch_peruser_test:190: should_create_anon_user_db_with_q4...[0.002 s] ok
       couch_peruser_test:191: should_create_anon_user_db_with_q4...ok
       couch_peruser_test:202: should_not_delete_user_db...[0.005 s] ok
       couch_peruser_test:213: should_delete_user_db...[0.003 s] ok
       couch_peruser_test:242: should_reflect_config_changes...[0.004 s] ok
       couch_peruser_test:249: should_add_user_to_db_admins...[0.002 s] ok
       couch_peruser_test:258: should_add_user_to_db_members...[0.002 s] ok
       couch_peruser_test:277: should_not_remove_existing_db_admins...ok
       couch_peruser_test:278: should_not_remove_existing_db_admins...ok
       couch_peruser_test:279: should_not_remove_existing_db_admins...ok
       couch_peruser_test:297: should_not_remove_existing_db_members...ok
       couch_peruser_test:298: should_not_remove_existing_db_members...ok
       couch_peruser_test:299: should_not_remove_existing_db_members...ok
       couch_peruser_test:325: should_remove_user_from_db_admins...ok
       couch_peruser_test:326: should_remove_user_from_db_admins...ok
       couch_peruser_test:327: should_remove_user_from_db_admins...ok
       couch_peruser_test:353: should_remove_user_from_db_members...ok
       couch_peruser_test:354: should_remove_user_from_db_members...ok
       couch_peruser_test:355: should_remove_user_from_db_members...ok
   [os_mon] cpu supervisor port (cpu_sup): Erlang has closed
       [done in 51.579 s]
     [done in 52.968 s]
   module 'couch_peruser'
   =======================================================
     All 25 tests passed.
   ==> rel (eunit)
   ==> asf2 (eunit)
   ```
   
   pr:
   
   ```
   ======================== EUnit ========================
   module 'couch_peruser_app'
   module 'couch_peruser_sup'
   module 'couch_peruser_test'
     couch_peruser test
   Application crypto was left running!
       couch_peruser_test:165: should_create_anon_user_db_with_default...[0.001 
s] ok
       couch_peruser_test:166: should_create_anon_user_db_with_default...ok
       couch_peruser_test:155: should_create_user_db_with_default...[0.002 s] ok
       couch_peruser_test:156: should_create_user_db_with_default...ok
       couch_peruser_test:178: should_create_user_db_with_q4...[0.003 s] ok
       couch_peruser_test:179: should_create_user_db_with_q4...ok
       couch_peruser_test:190: should_create_anon_user_db_with_q4...[0.002 s] ok
       couch_peruser_test:191: should_create_anon_user_db_with_q4...ok
       couch_peruser_test:203: should_not_delete_user_db...ok
       couch_peruser_test:203: should_not_delete_user_db...ok
       couch_peruser_test:215: should_delete_user_db...ok
       couch_peruser_test:215: should_delete_user_db...ok
       couch_peruser_test:246: should_reflect_config_changes...ok
       couch_peruser_test:247: should_reflect_config_changes...ok
       couch_peruser_test:248: should_reflect_config_changes...ok
       couch_peruser_test:249: should_reflect_config_changes...ok
       couch_peruser_test:250: should_reflect_config_changes...ok
       couch_peruser_test:251: should_reflect_config_changes...ok
       couch_peruser_test:260: should_add_user_to_db_admins...[0.002 s] ok
       couch_peruser_test:269: should_add_user_to_db_members...[0.002 s] ok
       couch_peruser_test:288: should_not_remove_existing_db_admins...ok
       couch_peruser_test:289: should_not_remove_existing_db_admins...ok
       couch_peruser_test:290: should_not_remove_existing_db_admins...ok
       couch_peruser_test:308: should_not_remove_existing_db_members...ok
       couch_peruser_test:309: should_not_remove_existing_db_members...ok
       couch_peruser_test:310: should_not_remove_existing_db_members...ok
       couch_peruser_test:339: should_remove_user_from_db_admins...ok
       couch_peruser_test:340: should_remove_user_from_db_admins...ok
       couch_peruser_test:341: should_remove_user_from_db_admins...ok
       couch_peruser_test:342: should_remove_user_from_db_admins...ok
       couch_peruser_test:343: should_remove_user_from_db_admins...ok
       couch_peruser_test:344: should_remove_user_from_db_admins...ok
       couch_peruser_test:373: should_remove_user_from_db_members...ok
       couch_peruser_test:374: should_remove_user_from_db_members...ok
       couch_peruser_test:375: should_remove_user_from_db_members...ok
       couch_peruser_test:376: should_remove_user_from_db_members...ok
       couch_peruser_test:377: should_remove_user_from_db_members...ok
       couch_peruser_test:378: should_remove_user_from_db_members...ok
   [os_mon] cpu supervisor port (cpu_sup): Erlang has closed
       [done in 51.886 s]
     [done in 53.309 s]
   module 'couch_peruser'
   =======================================================
     All 38 tests passed.
   ==> rel (eunit)
   ==> asf3 (eunit)
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to