Backend for getting dependencies need to update (#106)

Author: liyanhui1228

dashboard/dashboard_builder.py

@@ -66,6 +66,7 @@ def __init__(
         self.checker = checker
         self.store = store
         self.deprecated_deps = self.get_deprecated_deps()
+        self.dependency_to_update = self.get_dependencies_needed_to_update()
 
     def _is_py_version_incompatible(self, result):
         if result.status == compatibility_store.Status.INSTALL_ERROR:
@@ -105,7 +106,7 @@ def has_issues(self, p: package.Package) -> bool:
 
         return False
 
-    def get_deprecated_deps(self) -> Mapping[str, Tuple[List, bool]]:
+    def get_deprecated_deps(self) -> Mapping[str, List]:
         """
         Returns if there are deprecated dependencies for a
         given package as well as the list of deprecated deps for a package.
@@ -116,20 +117,29 @@ def get_deprecated_deps(self) -> Mapping[str, Tuple[List, bool]]:
 
         results = {}
         for item in deprecated_deps:
-            has_deprecated_deps = False
             (pkg_name, deps) = item[0]
-            if deps:
-                has_deprecated_deps = True
-            results[pkg_name] = (deps, has_deprecated_deps)
+            results[pkg_name] = deps
 
         return results
 
     def has_deprecated_deps(self, p: package.Package) -> bool:
-        return self.deprecated_deps[p.install_name][1]
+        return bool(self.deprecated_deps[p.install_name])
+
+    def get_dependencies_needed_to_update(self) -> Mapping[str, List]:
+        """
+        Returns a dict of package names together with the dependencies that
+        they need to update.
+        """
+        highlighter = dependency_highlighter.DependencyHighlighter(
+            py_version='3', checker=self.checker, store=self.store)
+        result = highlighter.check_packages(
+            packages=configs.PKG_LIST, max_workers=10)
+        return result
 
     def needs_update(self, p: package.Package) -> bool:
-        # Returns True if the given package needs update.
-        pass
+        """Returns whether the dependencies for a given package needs to
+        update."""
+        return bool(self.dependency_to_update[p.install_name])
 
     def get_statistics(self, packages):
         """Get the total number of packages that has issues."""

dashboard/grid-template.html

@@ -116,38 +116,39 @@
     {% for row_package in packages %}
     {% set row = loop.index0 %}
     <tr>
-        <td><font face="verdana">{{ row_package.friendly_name }}</font>{% if results.has_issues(row_package) %}
+        <td><font face="verdana">{{ row_package.friendly_name }}</font>
+            {% if results.has_issues(row_package) %}
                 {% set issues = results.deprecated_deps[row_package.friendly_name][0] %}
                 <img class="warning-img" class="dialog" onclick="showIssues('{{ row_package.friendly_name|safe }}', '{{ issues|replace('\'', '\\\'')|safe }}')" src="https://www.gstatic.com/images/icons/material/system/1x/warning_amber_18dp.png">
                 <div id="{{ row_package.friendly_name }}" class="dialog" title="{{ row_package.friendly_name }}"></div>
-            {% endif %}</td>
-        {% for col_package in packages %}
-        {% set column = loop.index0 %}
-        {% if column <= row %}
-            {% set result = results.get_result(row_package, col_package) %}
-            {% set cellName = row | string + '-' + column | string %}
-            {% set bgColorName = result.status_type | lower %}
-            {% if result.pairwise_compatibility_check | length > 0 %}
-                {% set pairwise_status = result.pairwise_compatibility_check[0].status %}
-                {% set pairwise_details = result.pairwise_compatibility_check[0].details %}
             {% endif %}
-            {% if result.self_compatibility_check | length > 0 %}
-                {% set self_1_status = result.self_compatibility_check[0].status %}
-                {% set self_1_details = result.self_compatibility_check[0].details %}
-                {% if result.self_compatibility_check | length > 1 %}
-                    {% set self_2_status = result.self_compatibility_check[1].status %}
-                    {% set self_2_details = result.self_compatibility_check[1].details %}
+        </td>
+        {% for col_package in packages %}
+            {% set column = loop.index0 %}
+            {% if column <= row %}
+                {% set result = results.get_result(row_package, col_package) %}
+                {% set cellName = row | string + '-' + column | string %}
+                {% set bgColorName = result.status_type | lower %}
+                {% if result.pairwise_compatibility_check | length > 0 %}
+                    {% set pairwise_status = result.pairwise_compatibility_check[0].status %}
+                    {% set pairwise_details = result.pairwise_compatibility_check[0].details %}
                 {% endif %}
+                {% if result.self_compatibility_check | length > 0 %}
+                    {% set self_1_status = result.self_compatibility_check[0].status %}
+                    {% set self_1_details = result.self_compatibility_check[0].details %}
+                    {% if result.self_compatibility_check | length > 1 %}
+                        {% set self_2_status = result.self_compatibility_check[1].status %}
+                        {% set self_2_details = result.self_compatibility_check[1].details %}
+                    {% endif %}
+                {% endif %}
+                <td class="{{ bgColorName }}" onclick="showDialog(
+                    '{{ cellName|safe }}', '{{ result.status_type|safe }}', '{{ self_1_status|safe }}', '{{ self_2_status|safe }}','{{ pairwise_status|safe }}', '{{ self_1_details|replace('\n', ' ')|safe}}', '{{ self_2_details|replace('\n', ' ')|safe}}', '{{ pairwise_details|replace('\n', ' ')|safe }}')">
+                    <div id="{{ cellName }}" class="dialog" title="{{ row_package.friendly_name }} and {{ col_package.friendly_name }}">
+                    </div>
+                </td>
+            {% else %}
+                <td><pre></pre></td>
             {% endif %}
-
-            <td class="{{ bgColorName }}" onclick="showDialog(
-                '{{ cellName|safe }}', '{{ result.status_type|safe }}', '{{ self_1_status|safe }}', '{{ self_2_status|safe }}','{{ pairwise_status|safe }}', '{{ self_1_details|replace('\n', ' ')|safe}}', '{{ self_2_details|replace('\n', ' ')|safe}}', '{{ pairwise_details|replace('\n', ' ')|safe }}')">
-                <div id="{{ cellName }}" class="dialog" title="{{ row_package.friendly_name }} and {{ col_package.friendly_name }}">
-                </div>
-            </td>
-        {% else %}
-            <td><pre></pre></td>
-        {% endif %}
         {% endfor %}
     </tr>
     {% endfor %}

dashboard/main-template.html

@@ -145,7 +145,7 @@ <h2 class="number">{{ statistics[3] }}</h2>
                 <div class="container">
                     <div class="row">
                         <div class="col-md-12">
-                            <h3 class="title-5 m-b-35">statistics</h3>
+                            <h3 class="title-5 m-b-35">Statistics</h3>
                         </div>
                     </div>
                     <div class="row">