Merge pull request finos#944 from fabiovincenzi/db-fix

fix: missing query parameter in file db

Author: JamieSlome

packages/git-proxy-cli/test/testCli.test.js

@@ -645,10 +645,12 @@ describe('test git-proxy-cli', function () {
 
   describe('test git-proxy-cli :: git push administration', function () {
     const pushId = `0000000000000000000000000000000000000000__${Date.now()}`;
+    const gitAccount = 'testGitAccount1';
 
     before(async function () {
       await helper.addRepoToDb(TEST_REPO_CONFIG);
-      await helper.addGitPushToDb(pushId, TEST_REPO);
+      await helper.addUserToDb('testuser1', 'testpassword', 'test@email.com', gitAccount);
+      await helper.addGitPushToDb(pushId, TEST_REPO, gitAccount);
     });
 
     it('attempt to ls should list existing push', async function () {

packages/git-proxy-cli/test/testCliUtils.js

@@ -168,16 +168,18 @@ async function addRepoToDb(newRepo, debug = false) {
  * Add a new git push record to the database.
  * @param {string} id The ID of the git push.
  * @param {string} repo The repository of the git push.
+ * @param {string} user The user who pushed the git push.
  * @param {boolean} debug Flag to enable logging for debugging.
  */
-async function addGitPushToDb(id, repo, debug = false) {
+async function addGitPushToDb(id, repo, user = null, debug = false) {
   const action = new actions.Action(
     id,
     'push', // type
     'get', // method
     Date.now(), // timestamp
     repo,
   );
+  action.user = user;
   const step = new steps.Step(
     'authBlock', // stepName
     false, // error

src/db/file/users.js

@@ -78,7 +78,7 @@ exports.updateUser = function (user) {
 exports.getUsers = function (query) {
   if (!query) query = {};
   return new Promise((resolve, reject) => {
-    db.find({}, (err, docs) => {
+    db.find(query, (err, docs) => {
       if (err) {
         reject(err);
       } else {