From e92f63775b349faae19d8df1c4cbdadbe5d12cec Mon Sep 17 00:00:00 2001 From: Lingamuneni Santhosh Siddhardha <103999924+Santhosh-Siddhardha@users.noreply.github.com> Date: Wed, 5 Jun 2024 23:10:30 +0530 Subject: [PATCH 1/2] Added lichess_test.py Added test file for lichess module --- src/test/lichess_test.py | 59 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 59 insertions(+) create mode 100644 src/test/lichess_test.py diff --git a/src/test/lichess_test.py b/src/test/lichess_test.py new file mode 100644 index 00000000..d646d1d3 --- /dev/null +++ b/src/test/lichess_test.py @@ -0,0 +1,59 @@ +import unittest +from src.scrape_up.lichess import LichessGames + + +class TestLichessGames(unittest.TestCase): + """ + | Methods | Details | + | ----------------------------- | -------------------------------------------------------------------------- | + | `.fetch_games()` | Fetch all the games data for the specified username. | + """ + def setUp(self): + """ + Initialize a LichessGames instance before each test method. + """ + self.username = "chess_player" # Example username + self.lichess_scraper = LichessGames(username=self.username) + + def test_fetch_games(self): + """ + Test the fetch_games() method. + """ + try: + games = self.lichess_scraper.fetch_games() + + # Check if games is a list of dictionaries + self.assertIsInstance(games, list) + for game in games: + self.assertIsInstance(game, dict) + self.assertIn("white_player", game) + self.assertIn("black_player", game) + self.assertIn("pgn", game) + + white_player = game["white_player"] + black_player = game["black_player"] + + self.assertIn("username", white_player) + self.assertIn("before_game_score", white_player) + self.assertIn("score_change", white_player) + + self.assertIn("username", black_player) + self.assertIn("before_game_score", black_player) + self.assertIn("score_change", black_player) + except: + return None + + def test_fetch_games_empty(self): + """ + Test fetch_games() method with a username that has no games. + """ + try: + self.lichess_scraper = LichessGames(username="non_existent_user") + games = self.lichess_scraper.fetch_games() + self.assertEqual(games, [], "Expected an empty list for a non-existent user") + except: + return None + + +if __name__ == "__main__": + unittest.main() From b0e42629c81e556ef368805beb86c1d51b0db100 Mon Sep 17 00:00:00 2001 From: Nikhil Raj Date: Thu, 6 Jun 2024 08:39:57 +0530 Subject: [PATCH 2/2] Check --- src/scrape_up/steam/steamScraper.py | 1 - src/test/lichess_test.py | 7 +++++-- src/test/quora_test.py | 3 --- src/test/swiggy_test.py | 2 -- 4 files changed, 5 insertions(+), 8 deletions(-) diff --git a/src/scrape_up/steam/steamScraper.py b/src/scrape_up/steam/steamScraper.py index dcbb3955..a44219db 100644 --- a/src/scrape_up/steam/steamScraper.py +++ b/src/scrape_up/steam/steamScraper.py @@ -240,7 +240,6 @@ def _scrape_page(self, url, filter, n0Games): if len(all_game_info) >= n0Games: break except requests.RequestException as e: - break return all_game_info diff --git a/src/test/lichess_test.py b/src/test/lichess_test.py index d646d1d3..a3446810 100644 --- a/src/test/lichess_test.py +++ b/src/test/lichess_test.py @@ -1,5 +1,5 @@ import unittest -from src.scrape_up.lichess import LichessGames +from scrape_up.lichess import LichessGames class TestLichessGames(unittest.TestCase): @@ -8,6 +8,7 @@ class TestLichessGames(unittest.TestCase): | ----------------------------- | -------------------------------------------------------------------------- | | `.fetch_games()` | Fetch all the games data for the specified username. | """ + def setUp(self): """ Initialize a LichessGames instance before each test method. @@ -50,7 +51,9 @@ def test_fetch_games_empty(self): try: self.lichess_scraper = LichessGames(username="non_existent_user") games = self.lichess_scraper.fetch_games() - self.assertEqual(games, [], "Expected an empty list for a non-existent user") + self.assertEqual( + games, [], "Expected an empty list for a non-existent user" + ) except: return None diff --git a/src/test/quora_test.py b/src/test/quora_test.py index f5d5289d..6ec84234 100644 --- a/src/test/quora_test.py +++ b/src/test/quora_test.py @@ -9,7 +9,6 @@ def setUp(self): def test_fetch_answers(self): try: - expected_answers = ["Accepted answer 1", "Suggested answer 1"] self.assertEqual( @@ -21,7 +20,6 @@ def test_fetch_answers(self): def test_get_by_query(self): try: - expected_answer = "Suggested answer 1" self.assertEqual( @@ -33,7 +31,6 @@ def test_get_by_query(self): def test_profile_details(self): try: - expected_profile = { "name": "Nikhil Raj", "url": "https://www.quora.com/profile/Nikhil-Raj", diff --git a/src/test/swiggy_test.py b/src/test/swiggy_test.py index c3939a0d..5d2779d7 100644 --- a/src/test/swiggy_test.py +++ b/src/test/swiggy_test.py @@ -18,7 +18,6 @@ def setUp(self): def test_get_restraunt_details(self): try: - expected_data = { "name": "Pizza Hut", "cuisine": "Pizzas", @@ -38,7 +37,6 @@ def test_get_restraunt_details(self): def test_get_restaurants(self): try: - expected_restaurants = [ { "Name": "Domino's Pizza",