changing directory structure for player profiles, this change is inco…

…mpatible with earlier versions - closes ryanb#3

features/levels.feature

@@ -8,20 +8,19 @@ Feature: Play levels
 
   Scenario: Pass first level, fail second level
     Given a profile named "Joe" on "beginner"
-    When I copy fixture "walking_player.rb" to "tmp/ruby-warrior/beginner-tower/level-001/player.rb"
+    When I copy fixture "walking_player.rb" to "tmp/ruby-warrior/joe-beginner/player.rb"
     And I run rubywarrior
     And I choose "Joe - beginner - level 1" for "profile"
     Then I answer "y" to "next level"
     And I should see "directory for the next level"
-    When I copy fixture "walking_player.rb" to "tmp/ruby-warrior/beginner-tower/level-002/player.rb"
     And I run rubywarrior
     And I choose "Joe - beginner - level 2" for "profile"
     And I answer "y" to "clues"
     Then I should see "warrior.feel.empty?"
 
   Scenario: Retry first level
     Given a profile named "Joe" on "beginner"
-    When I copy fixture "walking_player.rb" to "tmp/ruby-warrior/beginner-tower/level-001/player.rb"
+    When I copy fixture "walking_player.rb" to "tmp/ruby-warrior/joe-beginner/player.rb"
     And I run rubywarrior
     And I choose "Joe - beginner - level 1" for "profile"
     Then I answer "n" to "next level"
@@ -32,7 +31,7 @@ Feature: Play levels
   Scenario: Replay levels as epic when finishing last level
     When I copy fixture "short-tower" to "towers/short"
     Given a profile named "Bill" on "short"
-    When I copy fixture "walking_player.rb" to "tmp/ruby-warrior/short-tower/level-001/player.rb"
+    When I copy fixture "walking_player.rb" to "tmp/ruby-warrior/bill-short/player.rb"
     And I run rubywarrior
     And I choose "Bill - short - level 1" for "profile"
     Then I answer "y" to "next level"

lib/ruby_warrior/level.rb

@@ -10,7 +10,7 @@ def initialize(profile, number)
     end
 
     def player_path
-      @profile.player_path + "/level-" + @number.to_s.rjust(3, '0')
+      @profile.player_path
     end
 
     def load_path

lib/ruby_warrior/player_generator.rb

@@ -18,10 +18,7 @@ def previous_level
 
     # TODO refactor and test this method
     def generate
-      if File.exist? previous_level.player_path
-        FileUtils.cp_r(previous_level.player_path, level.player_path)
-        FileUtils.rm(level.player_path + '/README')
-      else
+      if level.number == 1
         FileUtils.mkdir_p(level.player_path)
         FileUtils.cp(templates_path + '/player.rb', level.player_path)
       end

lib/ruby_warrior/profile.rb

@@ -33,15 +33,11 @@ def self.load(path)
     end
 
     def player_path
-      Config.path_prefix + "/ruby-warrior/#{tower.name}-tower"
+      Config.path_prefix + "/ruby-warrior/#{directory_name}"
     end
 
-    def current_level_path
-      if epic?
-        player_path + "/epic"
-      else
-        player_path + "/level-" + level_number.to_s.rjust(3, '0')
-      end
+    def directory_name
+      [warrior_name.downcase.gsub(/[^a-z0-9]+/, '-'), tower.name].join('-')
     end
 
     def to_s

spec/ruby_warrior/level_spec.rb

@@ -28,9 +28,9 @@
     @level.description.should == 'foo'
   end
 
-  it "should have a player path from profile with level number in it" do
+  it "should have a player path from profile" do
     @profile.stubs(:player_path).returns('path/to/player')
-    @level.player_path.should == 'path/to/player/level-001'
+    @level.player_path.should == 'path/to/player'
   end
 
   it "should have a load path from profile tower with level number in it" do

spec/ruby_warrior/profile_spec.rb

@@ -83,6 +83,7 @@
 
   describe "with tower path" do
     before(:each) do
+      @profile.warrior_name = "John Smith"
       @profile.tower_path = 'path/to/tower'
     end
 
@@ -106,16 +107,7 @@
     end
 
     it "should guess at the player path" do
-      @profile.player_path.should == './ruby-warrior/tower-tower'
-    end
-
-    it "should append level dir to player path" do
-      @profile.current_level_path.should == './ruby-warrior/tower-tower/level-000'
-    end
-
-    it "should use 'epic' as level directory" do
-      @profile.enable_epic_mode
-      @profile.current_level_path.should == './ruby-warrior/tower-tower/epic'
+      @profile.player_path.should == './ruby-warrior/john-smith-tower'
     end
 
     it "should load tower from path" do