Merge pull request #161 from nexB/increase-rescan-priority

Increase priority on rescanned packages
aboutcode-org · Aug 8, 2023 · c775297 · c775297
2 parents 5258a92 + 7c0ce60
commit c775297
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/minecode/management/commands/process_scans.py b/minecode/management/commands/process_scans.py
@@ -145,6 +145,7 @@ def process_scan(cls, scannable_uri, get_scan_info_save_loc='', get_scan_data_sa
                 scannable_uri.scan_status = ScannableURI.SCAN_INDEXED
                 if rescan:
                     scannable_uri.rescan = False
+                    scannable_uri.priority = 0
 
             except Exception as e:
                 error_message = str(e) + '\n'

diff --git a/minecode/models.py b/minecode/models.py
@@ -773,6 +773,7 @@ def rescan(self):
         self.index_error = None
         self.scan_uuid = None
         self.scan_request_date = None
+        self.priority = 100
         self.save()
 
 

diff --git a/packagedb/tests/test_api.py b/packagedb/tests/test_api.py
@@ -520,6 +520,7 @@ def setUp(self):
 
     def test_reindex_package(self):
         self.assertEqual(False, self.scannableuri.rescan_uri)
+        self.assertEqual(0, self.scannableuri.priority)
         self.assertEqual(self.scan_uuid, self.scannableuri.scan_uuid)
         self.assertEqual('error', self.scannableuri.scan_error)
         self.assertEqual('error', self.scannableuri.index_error)
@@ -528,19 +529,22 @@ def test_reindex_package(self):
         self.assertEqual('pkg:maven/sample/baz@90.12 has been queued for reindexing', response.data['status'])
         self.scannableuri.refresh_from_db()
         self.assertEqual(True, self.scannableuri.rescan_uri)
+        self.assertEqual(100, self.scannableuri.priority)
         self.assertEqual(None, self.scannableuri.scan_uuid)
         self.assertEqual(None, self.scannableuri.scan_error)
         self.assertEqual(None, self.scannableuri.index_error)
         self.assertEqual(None, self.scannableuri.scan_request_date)
 
     def test_reindex_packages_basic(self):
         self.assertEqual(False, self.scannableuri.rescan_uri)
+        self.assertEqual(0, self.scannableuri.priority)
         self.assertEqual(self.scan_uuid, self.scannableuri.scan_uuid)
         self.assertEqual('error', self.scannableuri.scan_error)
         self.assertEqual('error', self.scannableuri.index_error)
         self.assertEqual(self.scan_request_date, self.scannableuri.scan_request_date)
 
         self.assertEqual(False, self.scannableuri2.rescan_uri)
+        self.assertEqual(0, self.scannableuri2.priority)
         self.assertEqual(self.scan_uuid2, self.scannableuri2.scan_uuid)
         self.assertEqual('error', self.scannableuri2.scan_error)
         self.assertEqual('error', self.scannableuri2.index_error)
@@ -563,13 +567,15 @@ def test_reindex_packages_basic(self):
 
         self.scannableuri.refresh_from_db()
         self.assertEqual(True, self.scannableuri.rescan_uri)
+        self.assertEqual(100, self.scannableuri.priority)
         self.assertEqual(None, self.scannableuri.scan_uuid)
         self.assertEqual(None, self.scannableuri.scan_error)
         self.assertEqual(None, self.scannableuri.index_error)
         self.assertEqual(None, self.scannableuri.scan_request_date)
 
         self.scannableuri2.refresh_from_db()
         self.assertEqual(True, self.scannableuri2.rescan_uri)
+        self.assertEqual(100, self.scannableuri.priority)
         self.assertEqual(None, self.scannableuri2.scan_uuid)
         self.assertEqual(None, self.scannableuri2.scan_error)
         self.assertEqual(None, self.scannableuri2.index_error)