change imports track Mathics3 changes

Overall, modularity is better in Mathics3 so we
don't have to import pattern_object inside methods anymore.

Author: rocky

pymathics/graph/base.py

@@ -15,6 +15,7 @@
 from mathics.core.convert.expression import ListExpression, from_python, to_mathics_list
 from mathics.core.element import BaseElement
 from mathics.core.expression import Expression
+from mathics.core.pattern import pattern_objects
 from mathics.core.symbols import Symbol, SymbolList, SymbolTrue
 from mathics.core.systemsymbols import (
     SymbolBlank,
@@ -853,7 +854,6 @@ class AdjacencyList(_NetworkXBuiltin):
     summary_text = "list the adjacent vertices"
 
     def _retrieve(self, graph, what, neighbors, expression, evaluation):
-        from mathics.builtin import pattern_objects
 
         if what.get_head_name() in pattern_objects:
             collected = set()
@@ -1426,7 +1426,6 @@ def eval(self, graph, what, expression, evaluation, options) -> Optional[Graph]:
         "VertexDelete[graph_, what_, OptionsPattern[VertexDelete]]"
         graph = self._build_graph(graph, evaluation, options, expression)
         if graph:
-            from mathics.builtin import pattern_objects
 
             head_name = what.get_head_name()
             if head_name in pattern_objects:
@@ -1567,7 +1566,6 @@ def eval(self, graph, what, expression, evaluation, options) -> Optional[Graph]:
         "EdgeDelete[graph_, what_, OptionsPattern[EdgeDelete]]"
         graph = self._build_graph(graph, evaluation, options, expression)
         if graph:
-            from mathics.builtin import pattern_objects
 
             head_name = what.get_head_name()
             if head_name in pattern_objects:

test/consistency-and-style/test_summary_text.py

@@ -7,7 +7,7 @@
 import pytest
 
 from pymathics.graph import __file__ as module_initfile_path
-from mathics.builtin import name_is_builtin_symbol
+from mathics.core.load_builtin import name_is_builtin_symbol
 from mathics.builtin.base import Builtin
 from mathics.doc.common_doc import skip_doc
 
@@ -70,8 +70,7 @@
 module_subdirs = tuple()
 
 __py_files__ = [
-    osp.basename(f[0:-3])
-    for f in glob.glob(osp.join(module_path, "[a-z]*.py"))
+    osp.basename(f[0:-3]) for f in glob.glob(osp.join(module_path, "[a-z]*.py"))
 ]
 
 
@@ -124,7 +123,6 @@ def check_grammar(text: str):
     return False
 
 
-
 def check_well_formatted_docstring(docstr: str, instance: Builtin, module_name: str):
     assert (
         docstr.count("<dl>") >= 1