Tobias47n9e has uploaded a new change for review. https://gerrit.wikimedia.org/r/296818
Change subject: Rename TestContainer to Container ...................................................................... Rename TestContainer to Container The automatic test discovery will try to run the tests in any class that starts with test. The TestContainer class does not contain any tests and should therefore be just called Container to increase the compatibility with pytest. Change-Id: Ia76cbdeb4591e6f8b65e03e0928ab793e7b6e2fb --- M tests/sparql_tests.py 1 file changed, 6 insertions(+), 6 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/pywikibot/core refs/changes/18/296818/1 diff --git a/tests/sparql_tests.py b/tests/sparql_tests.py index f7c2f5c..d86368e 100644 --- a/tests/sparql_tests.py +++ b/tests/sparql_tests.py @@ -74,7 +74,7 @@ """ -class TestContainer(object): +class Container(object): """Simple test container for return values.""" def __init__(self, value): @@ -90,7 +90,7 @@ @patch.object(sparql.http, 'fetch') def testQuerySelect(self, mock_method): """Test SELECT query.""" - mock_method.return_value = TestContainer(SQL_RESPONSE) + mock_method.return_value = Container(SQL_RESPONSE) q = sparql.SparqlQuery() res = q.select('SELECT * WHERE { ?x ?y ?z }') self.assertIsInstance(res, list, 'Result is not a list') @@ -108,7 +108,7 @@ @patch.object(sparql.http, 'fetch') def testQuerySelectFull(self, mock_method): """Test SELECT query with full data.""" - mock_method.return_value = TestContainer(SQL_RESPONSE) + mock_method.return_value = Container(SQL_RESPONSE) q = sparql.SparqlQuery() res = q.select('SELECT * WHERE { ?x ?y ?z }', full_data=True) self.assertIsInstance(res, list, 'Result is not a list') @@ -130,7 +130,7 @@ @patch.object(sparql.http, 'fetch') def testGetItems(self, mock_method): """Test item list retrieval via SPARQL.""" - mock_method.return_value = TestContainer(SQL_RESPONSE) + mock_method.return_value = Container(SQL_RESPONSE) q = sparql.SparqlQuery() res = q.get_items('SELECT * WHERE { ?x ?y ?z }', 'cat') self.assertSetEqual(res, set(['Q498787', 'Q677525'])) @@ -138,13 +138,13 @@ @patch.object(sparql.http, 'fetch') def testQueryAsk(self, mock_method): """Test ASK query.""" - mock_method.return_value = TestContainer(RESPONSE_TRUE) + mock_method.return_value = Container(RESPONSE_TRUE) q = sparql.SparqlQuery() res = q.ask('ASK { ?x ?y ?z }') self.assertTrue(res) - mock_method.return_value = TestContainer(RESPONSE_FALSE) + mock_method.return_value = Container(RESPONSE_FALSE) res = q.ask('ASK { ?x ?y ?z }') self.assertFalse(res) -- To view, visit https://gerrit.wikimedia.org/r/296818 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ia76cbdeb4591e6f8b65e03e0928ab793e7b6e2fb Gerrit-PatchSet: 1 Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-Owner: Tobias47n9e <tobias47...@gmail.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits