Repository: great-expectations/great_expectations

Issue:
Use cleaner solution for non-truncating division in python 2
Prefer `from __future__ import division` to `1.*x/y`


Discussion:
Done inline during improve_partitions push.

Resolving patch:
diff --git a/great_expectations/cli/toolkit.py b/great_expectations/cli/toolkit.py
--- a/great_expectations/cli/toolkit.py
+++ b/great_expectations/cli/toolkit.py
@@ -388,7 +388,7 @@ def run_checkpoint(
     usage_event: str,
 ) -> CheckpointResult:
     """Run a checkpoint or raise helpful errors."""
-    failure_message: str = "Exception occurred while running checkpoint."
+    failure_message: str = "Exception occurred while running Checkpoint."
     validate_checkpoint(
         context=context,
         checkpoint_name=checkpoint_name,
diff --git a/great_expectations/data_context/types/base.py b/great_expectations/data_context/types/base.py
--- a/great_expectations/data_context/types/base.py
+++ b/great_expectations/data_context/types/base.py
@@ -80,10 +80,6 @@ def _get_schema_instance(cls) -> Schema:
     @classmethod
     def from_commented_map(cls, commented_map: CommentedMap):
         try:
-            # TODO: <Alex>ALEX</Alex>
-            # config: dict = cls._get_schema_instance().load(commented_map)
-            # return cls.get_config_class()(commented_map=commented_map, **config)
-            # TODO: <Alex>ALEX</Alex>
             config: Union[dict, BaseYamlConfig]
             config = cls._get_schema_instance().load(commented_map)
             if isinstance(config, dict):


Test patch:
diff --git a/tests/cli/test_checkpoint.py b/tests/cli/test_checkpoint.py
--- a/tests/cli/test_checkpoint.py
+++ b/tests/cli/test_checkpoint.py
@@ -940,7 +940,7 @@ def test_checkpoint_run_on_checkpoint_with_empty_suite_list_raises_error(
     assert result.exit_code == 1
 
     stdout: str = result.stdout
-    assert "Exception occurred while running checkpoint" in stdout
+    assert "Exception occurred while running Checkpoint" in stdout
     assert (
         "of Checkpoint 'no_suite': validation expectation_suite_name must be specified"
         in stdout
@@ -1943,7 +1943,7 @@ def test_checkpoint_run_happy_path_with_failed_validation_due_to_bad_data_pandas
     assert result.exit_code == 1
 
     stdout: str = result.stdout
-    assert "Exception occurred while running checkpoint." in stdout
+    assert "Exception occurred while running Checkpoint." in stdout
     assert 'Error: The column "Name" in BatchData does not exist...' in stdout
 
     assert mock_emit.call_count == 4
@@ -2065,7 +2065,7 @@ def test_checkpoint_run_happy_path_with_failed_validation_due_to_bad_data_sql(
     assert result.exit_code == 1
 
     stdout: str = result.stdout
-    assert "Exception occurred while running checkpoint." in stdout
+    assert "Exception occurred while running Checkpoint." in stdout
     assert 'Error: The column "Name" in BatchData does not exist...' in stdout
 
     assert mock_emit.call_count == 4
@@ -2198,7 +2198,7 @@ def test_checkpoint_run_happy_path_with_failed_validation_due_to_bad_data_spark(
     assert result.exit_code == 1
 
     stdout: str = result.stdout
-    assert "Exception occurred while running checkpoint." in stdout
+    assert "Exception occurred while running Checkpoint." in stdout
     assert 'Error: The column "Name" in BatchData does not exist...' in stdout
 
     assert mock_emit.call_count == 4