Tidied up

test/lib/deb_importer_test.rb

@@ -9,28 +9,6 @@ include DebImporter
 class PackagesHelperTest < ActionView::TestCase
 
   test "should be able to parse local Debian repository test files" do
-
-    # How many packages of each component and architecture we expect
-    # when parsing the text files in lib/tasks/files/...
-    expected_package_counts = {
-      main: {
-        amd64: 8320,
-        i386: 8082
-        },
-        restricted: {
-          amd64: 21,
-          i386: 21
-          },
-          universe: {
-            amd64: 29406,
-            i386: 29471
-            },
-            multiverse: {
-              amd64: 672,
-              i386: 685
-            }
-          }
-
     Rails.logger = Logger.new(STDOUT)
     Rails.logger.level = Logger::ERROR
     # Rails.logger.level = Logger::INFO
@@ -45,12 +23,13 @@ class PackagesHelperTest < ActionView::TestCase
     release = DebImporter::Release.new(repository[:url])
     assert_equal release.components, 'main restricted universe multiverse'
 
+    # Packages enumerator must be restarted to start from position 1
     packages = release.packages('main', 'amd64')
     assert_instance_of Enumerator, packages
     first_package = packages.first
     assert_equal 'account-plugin-aim', first_package[:Package]
 
-    # .count must be checked seperately because it changes the position in the enumerator
+    # Packages enumerator must be restarted to start from position 1
     packages = release.packages('main', 'amd64')
     assert_equal 8320, packages.count
   end # test