The 8.x-1.x version of the module does not (yet) include the drush commands provided in the 7.x-1.x version:

  • cas-user-add-role
  • cas-user-create

Comments

joegraduate created an issue. See original summary.

bkosborne’s picture

Status: Active » Needs review
StatusFileSize
new1.93 KB

Ok so I don't think we need to port over those two commands exactly:

1) cas-user-add-role - There's a drush command for adding roles to users, don't see why we need this.
2) cas-user-create - I think this does too much. Drush already has a command for creating new users. All we need is the ability to associate that user with some CAS username.

I think we're fine adding just one command which associates an existing account with CAS.

Status: Needs review » Needs work

The last submitted patch, 2: cas-drush-commands-2950946.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

bkosborne’s picture

Ugh... no idea why those tests are failing.

bkosborne’s picture

Status: Needs work » Needs review
StatusFileSize
new308 bytes

Status: Needs review » Needs work

The last submitted patch, 5: cas-drush-commands-2950946-5.patch, failed testing. View results

bkosborne’s picture

Status: Needs work » Needs review
StatusFileSize
new1.88 KB

Status: Needs review » Needs work

The last submitted patch, 7: cas-drush-commands-2950946-7.patch, failed testing. View results

bkosborne’s picture

Status: Needs work » Needs review
StatusFileSize
new0 bytes

Trying again after I removed the branch alias from upstream.

Status: Needs review » Needs work

The last submitted patch, 9: cas-drush-commands-2950946-9.patch, failed testing. View results

bkosborne’s picture

Status: Needs work » Needs review
StatusFileSize
new1.96 KB

oops

Status: Needs review » Needs work

The last submitted patch, 11: cas-drush-commands-2950946-11.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

bkosborne’s picture

Status: Needs work » Fixed

These tests pass locally. I'm 95% sure these failures are related to the fact that we're modifying the composer.json file, and it's tripping up the test runner somehow. It seems like it's not pulling down the external auth dependency.

  • bkosborne committed 9eebbc4 on 8.x-1.x
    Issue #2950946 by bkosborne: Port drush commands to D8
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.