refactor(token-handler): remove redundant PKCE check for client_id retrieval and rename test description

Author: dsschiramm

lib/handlers/token-handler.js

@@ -190,12 +190,6 @@ class TokenHandler {
       };
     }
 
-    if (pkce.isPKCERequest({ grantType, codeVerifier })) {
-      if (request.body.client_id) {
-        return { clientId: request.body.client_id };
-      }
-    }
-
     if (request.body.client_id) {
       return { clientId: request.body.client_id };
     }

test/unit/handlers/token-handler_test.js

@@ -45,7 +45,7 @@ describe('TokenHandler', function () {
         .catch(should.fail);
     });
 
-    it('should call `model.getClient()` with no secret is provided (public client)', function () {
+    it('should call `model.getClient()` when no client secret is provided (public client)', function () {
       const model = Model.from({
         getClient: sinon.stub().returns({ grants: ['authorization_code'], type: 'public' }),
         saveToken: function () {},