-More strict test for job id

-TODO:Revise code before pushing -TODO:Make sure to update readme for Docker docs
-TODO:Make sure to update readme for Docker docs
-TODO:Update readme on new GlassDoor scrapers

Author: thebigG

tests/test_indeed.py

@@ -11,14 +11,15 @@ def get_number_of_pages_mock(mock_soup, mock_max):
     """
     mock get_number_of_pages to ensure we only scrape no more than 1 page.
     """
-    return 	
+    return 1
 
 
 #test the correctness of search_tems since our tests depend on it    
 
 def test_search_terms(init_scraper):
     indeed = init_scraper('indeed')
-        
+    assert indeed.search_terms == { 'region': {'province':'ON', 
+    'city':'waterloo', 'domain':'ca', 'radius':25}, 'keywords':['Python']}
 
 @pytest.mark.parametrize('search_terms_config', search_term_configs)
 class TestClass():
@@ -165,7 +166,6 @@ def test_get_id(self, setup_scraper, search_terms_config):
             try:
                 job['id'] = provider.get_id(soup)
             except:
-                job['id'] = ''
                 assert False
         assert True
 
@@ -183,9 +183,6 @@ def test_get_link(self, setup_scraper, search_terms_config):
                 job['link'] = provider.get_link(job['id'])
             except AttributeError:
                 continue
-            # TODO:Maybe the testing for links could be made more realiable
-            # by having statistics on them such as '8/10 links passed' given that the link is probably the
-            # most essential piece of data for users
             if(0 < len(job['link'])):
                 assert True
                 return