summaryrefslogtreecommitdiffstats
path: root/pw/tests
Commit message (Collapse)AuthorAgeFilesLines
* Fix err pointer not initialized to NULL resultingBaptiste Daroussin2015-08-211-0/+15
| | | | Reported by: "O. Hartmann" <ohartman@zedat.fu-berlin.de>
* Fix typo in regression testBaptiste Daroussin2015-08-211-1/+1
|
* Fix /home symlink creationBaptiste Daroussin2015-08-211-0/+13
| | | | Add regression test about it
* Fix useradd regression:Baptiste Daroussin2015-08-211-2/+1
| | | | | | | | Readd the function to create the parents home directory if it does not exists. if it is only a directory at the top level of the hierarchy symlink it into /usr as it used to be done before. Reported by: kevlo, adrian
* Regression: fix pw usermod -w xxxBaptiste Daroussin2015-08-141-0/+14
| | | | Reported by: gjb
* Add regression tests for a bug reported in stable/10Baptiste Daroussin2015-08-051-0/+9
| | | | | While pw(8) on head is not affected it is worth adding more regression tests ensuring this bug will not happen unnoticed in the futur
* Convert the year used for regression test fro 2043 to 2037Baptiste Daroussin2015-08-021-11/+11
| | | | This makes the regression tests pass on systems where time_t is 32bits
* Add regression tests about adding already existsing groups/usersBaptiste Daroussin2015-08-022-0/+22
|
* Rewrite parsing subcommands arguments of pw(8)Baptiste Daroussin2015-08-024-8/+60
| | | | | | | | | | | | | Now each subcommands checks its arguments in a dedicated functions. This helps improving input validation, code readability/maintainability While here: - Add a -y option to pw userdel/usermod so it can maintain NIS servers if nispasswd is not defined in pw.conf(5) - Allow pw -r <rootdir> to remove directory with userdel -r - Fix bug when renaming a user which was not renaming the user name it groups it is a member of. - Only parse pw.conf(5) when needed.
* Remove netbsd tests on pw(8)Baptiste Daroussin2015-08-021-11/+0
| | | | | | First they are redundant with the tests we currently have on pw(8) Second they to modify the host database instead of being self contained withing the test directory
* Check uid/gid used when creating a user/group are not larger than ↵Baptiste Daroussin2015-07-283-0/+24
| | | | | | | UID_MAX/GID_MAX PR: 173977 Reported by: nvass@gmx.com
* when -n is passed to any pw subcommand it is always expected to be considered asBaptiste Daroussin2015-07-281-0/+10
| | | | | | | | a name so do not try to convert it to an id if it is a numeric value PR: 31933 Reported by: ted@impulse.net Sponsored by: gandi.net
* Add regression tests to ensure we keep allowing creating users with uid0Baptiste Daroussin2015-07-131-0/+11
|
* Ensure skeldir is abolute path (relatively to the rootdir)Baptiste Daroussin2015-07-122-2/+2
|
* pw -R <rootdir> userdel can now cleanup installationBaptiste Daroussin2015-07-121-0/+20
| | | | | | | Rewrite rm_r to use *at function, allowing to remove home directories along with users. only crontabs and at(1) installation are not removed Relnotes: yes
* Rework the home directory creation and copy or the skel content to use *atBaptiste Daroussin2015-07-121-0/+25
| | | | | | | | | functions This allows to simplify the code a bit for -R by not having to keep modifying path and also prepare the code to improve support -R in userdel While here, add regression tests for the functionality
* Add minimum regression tests for pw -RBaptiste Daroussin2015-07-122-1/+31
|
* Rework groupmod modification:Baptiste Daroussin2015-07-111-0/+27
| | | | | Use gr_add(3) when possible to avoid code duplication. Use a simpler logic to delete members of a group
* Do not try to set password on group if the group is added as a consequence ofBaptiste Daroussin2015-07-091-0/+10
| | | | | | of creating a user (regression from r285136) Reported by: Fabian Keil <fk@fabiankeil.de>
* pw: fail if an invalid entry is found while parsing master.passwd and groupBaptiste Daroussin2015-07-071-0/+26
| | | | | | PR: 198554 Reported by: diaran <fbsd@centraltech.co.uk> MFC after: 2 days
* Also validate hours via strptime_l(3)Baptiste Daroussin2015-07-051-0/+6
| | | | | Simplify the code, by only using one parser, ensure the dates (hours and dates) are valid
* Validate expiration datesBaptiste Daroussin2015-07-051-0/+26
| | | | Use strptime_l(3) to validate the dates provided in input
* Validate input of pw usermod -h and pwusermod -HBaptiste Daroussin2015-07-041-0/+37
| | | | | | | Push the code that set the password into a separate function to improve readability Add regression tests about pw usermod -h and pw usermod -H
* Fix setting uid/gid min/max via pwBaptiste Daroussin2015-06-073-1/+74
|
* Fix generating configuration fileBaptiste Daroussin2015-06-073-1/+80
|
* In case of rename validate the length of the new nameBaptiste Daroussin2015-06-072-3/+35
| | | | Check early that the new name fits MAXLOGNAME and store it in pwconf
* Refactor input validationBaptiste Daroussin2015-06-071-1/+9
| | | | | Mutualize code to validate inputs of both 'user' and 'group' command Test that the input name fits into MAXLOGNAME
* Fix a race condition by allowing up to 5 seconds of difference between the ↵Brad Davis2015-02-241-4/+8
| | | | | | | time stamps. PR: 197861 Approved by: will
* Add tests for account and password expiration.Brad Davis2015-02-171-0/+111
| | | | Approved by: will
* Really fix the pw_usernext tests.Brad Davis2015-02-151-12/+15
| | | | | PR: 197612 Approved by: will
* Remove an extra curly bracket that was causing intermittent failures.Brad Davis2015-02-141-1/+1
| | | | | | PR: 197612 Submitted by: Robert O'Niel <oneil.rs@gmail.com> Approved by: will
* Add tests for `pw usernext'.Brad Davis2015-02-092-1/+44
| | | | | | PR: 197120 Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will
* Revert r277652Baptiste Daroussin2015-01-263-41/+1
| | | | | | uid and gid are never and should never be negative. The pw(8) manpage clearly states the -u and -g arguments are for uids/gids, hence using negative values is abusing a bug in former versions of pw(8)
* Allow negative numbers in -u and -g optionsBaptiste Daroussin2015-01-243-1/+41
| | | | | PR: 196514 MFC after: 1 week
* Add tests for pw -NBrad Davis2014-12-162-0/+89
| | | | | | PR: 150449 Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will
* Restructure to make it easier to extend in the futureBrad Davis2014-12-091-6/+10
| | | | | Reviewed by: will Suggested by: ngie
* Add some tests for user modification. [1]Brad Davis2014-12-092-1/+59
| | | | | | | | Fix a missing test in the Makefile from my previous commit. PR: 195834 [1] Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will
* Break out the tests into a file per command and clean up some long linesBrad Davis2014-12-095-19/+33
| | | | Approved by: will
* Add more tests for pw(8) useradd.Brad Davis2014-12-092-1/+42
| | | | | | PR: 195832 Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will
* Replace the pw(1) calls with a variable, to make it easier to to change ↵Brad Davis2014-11-194-32/+35
| | | | | | across all the tests as needed. Reviewed by: will
* Clean up some languageBrad Davis2014-11-172-18/+18
| | | | Reviewed by: will
* Add a test for locking and unlocking user accountsBrad Davis2014-11-152-1/+23
| | | | | Submitted by: Robert O'Neil <oneil.rs@gmail.com> Reviewed by: will
* Fix wrong message when using pw -V with a non existent directoryBaptiste Daroussin2014-11-122-1/+19
| | | | | | | Add a regression test about it PR: 194971 Submitted by: Freddy DISSAUX <bugzilla@dsx.bsdsx.fr>
* Require root when running the pw_delete and pw_modify test programs (pw(8)Enji Cooper2014-11-031-0/+3
| | | | requires it)
* Integrate usr.sbin/useradd/t_useradd.sh from NetBSD into FreeBSD as pw_testEnji Cooper2014-11-031-0/+11
| | | | | | | | | - Suffix useradd/userdel commands with pw - Remove the atf_expect_fail for bin/39546 Phabric: D921 Reviewed by: brd Sponsored by: EMC / Isilon Storage Division
* Fix renaming a group via the gr_copy functionBaptiste Daroussin2014-10-281-0/+14
| | | | | | | | | | Add a regression test to pw(8) because the bug was discovered via using: pw groupmod PR: 187189 Reported by: mcdouga9@egr.msu.edu Tested by: mcdouga9@egr.msu.edu Patch by: Marc de la Gueronniere
* Ensure pw userdel -u <invalid> do not try to remove rootBaptiste Daroussin2014-10-281-0/+11
| | | | | | Check the uid passed is actually a number as early as possible MFC after: 1 week
* Fix atf syntaxBaptiste Daroussin2014-10-281-1/+1
|
* Do not delete the group wheel when bad argument is passed to pw groupdel -gBaptiste Daroussin2014-10-281-0/+12
| | | | | | | | | | | | Check that the -g argument is actually a number, if not report an error. This argument is converted without checking with atoi(3) later so without this check it converts any alpha entries into 0 meaning it deletes the group wheel Add a regression test about it PR: 90114 Reported by: bkoenig@cs.tu-berlin.de MFC after: 1 week
* Fix a regression in pw usermod -G listBaptiste Daroussin2014-10-281-0/+21
| | | | | | | | | | The user was perperly adding the to different groups from "list" but was not removed from the other groups it could have belong to. While here add a regression test about this bug PR: 185666 Reported by: sub.mesa@gmail.com MFC after: 1 week