feat: make allow_access behave closer to htpasswd default auth plugin

- add new unit test for empty access level value config

Author: dlouzan

test/unit/gitlab.spec.js

@@ -92,6 +92,24 @@ describe('Gitlab Auth Plugin Unit Tests', () => {
     verdaccioGitlab.allow_access(config.remoteUser, _package, cb);
   });
 
+  test('should allow access to package when access level is empty (default = $all)', done => {
+    const verdaccioGitlab: VerdaccioGitlab = new VerdaccioGitlab(config.verdaccioGitlabConfig, config.options);
+    const _package: VerdaccioGitlabPackageAccess = {
+      name: config.user,
+      access: undefined,
+      gitlab: true
+    };
+
+    const cb: Callback = (err, data) => {
+      expect(err).toBeFalsy();
+      // false allows the plugin chain to continue
+      expect(data).toBe(true);
+      done();
+    };
+
+    verdaccioGitlab.allow_access(config.remoteUser, _package, cb);
+  });
+
   test('should deny access to package based on unauthenticated', done => {
     const verdaccioGitlab: VerdaccioGitlab = new VerdaccioGitlab(config.verdaccioGitlabConfig, config.options);
     const unauthenticatedUser: RemoteUser = {