use a default name for spiders constructed without names

scrapy/command/commands/fetch.py

@@ -41,7 +41,7 @@ class Command(ScrapyCommand):
                 log.msg("Could not find spider: %s" % opts.spider, log.ERROR)
         else:
             spider = scrapymanager._create_spider_for_request(request, \
-                BaseSpider('default'))
+                BaseSpider())
 
         scrapymanager.crawl_request(request, spider)
         scrapymanager.start()

scrapy/shell.py

@@ -55,7 +55,7 @@ class Shell(object):
             request = Request(url)
 
         spider = scrapymanager._create_spider_for_request(request, \
-            BaseSpider('default'), log_multiple=True)
+            BaseSpider(), log_multiple=True)
 
         print "Fetching %s..." % request
         response = threads.blockingCallFromThread(reactor, scrapyengine.schedule, \

scrapy/spider/models.py

@@ -55,7 +55,7 @@ class BaseSpider(object_ref):
         if not getattr(self, 'extra_domain_names', None):
             self.extra_domain_names = self.allowed_domains
         if not self.name:
-            raise ValueError("%s must have a name" % type(self).__name__)
+            self.name = 'default'
 
     def log(self, message, level=log.DEBUG):
         """Log the given messages at the given log level. Always use this

scrapy/tests/test_spider.py

@@ -66,9 +66,10 @@ class BaseSpiderTest(unittest.TestCase):
 
     def test_spider_without_name(self):
         """Constructor arguments are assigned to spider attributes"""
-        spider = self.spider_class('example.com')
-        self.assertRaises(ValueError, self.spider_class)
-        self.assertRaises(ValueError, self.spider_class, somearg='foo')
+        spider = self.spider_class()
+        self.assertEqual(spider.name, 'default')
+        spider = self.spider_class(foo='bar')
+        self.assertEqual(spider.foo, 'bar')
 
 
 class InitSpiderTest(BaseSpiderTest):