Merge pull request #2355 from openstax/fix_exercise_edit_urls

Fixed the exercise edit URLs (for admins)

Author: Dantemss

app/routines/export_exercise_exclusions.rb

@@ -62,7 +62,7 @@ def page_urls_by_pages
   def exercise_urls_by_exercise_numbers
     @exercise_urls_by_exercise_numbers ||= {}.tap do |hash|
       all_excluded_exercise_numbers.each do |number|
-        hash[number] = OpenStax::Exercises::V1.uri_for("/exercises/#{number}").to_s
+        hash[number] = OpenStax::Exercises::V1.uri_for("/exercise/#{number}").to_s
       end
     end
   end

lib/openstax/exercises/v1/exercise.rb

@@ -40,7 +40,7 @@ def uid
   end
 
   def url
-    @url ||= "#{@server_url}/exercises/#{uid}"
+    @url ||= "#{@server_url}/exercise/#{uid}"
   end
 
   def nickname

spec/fixtures/box/excluded_exercises_stats_by_course_20170925T213441Z.csv

@@ -1,2 +1,2 @@
 Course ID,Course Name,Course Type,Teachers,# Exclusions,Excluded Exercise Numbers,Excluded Exercise URLs,Exclusion Timestamps,CNX Book Title,CNX Book UUID,CNX Book Locations,CNX Page UUIDs,CNX Page URLs
-66,cum nobis consectetur,Tutor,Bob Martin,4,"-4, -3, -2, -1","https://exercises-dev.openstax.org/exercises/-4, https://exercises-dev.openstax.org/exercises/-3, https://exercises-dev.openstax.org/exercises/-2, https://exercises-dev.openstax.org/exercises/-1","2017-09-25T21:34:40.085Z, 2017-09-25T21:34:40.096Z, 2017-09-25T21:34:40.107Z, 2017-09-25T21:34:40.117Z",saepe nam mollitia,929ad821-5801-4272-9186-ffb0c047ec08,"1.1, 1.2, 1.2, null","3480b873-4599-4e23-b139-afb532da162b, 5329c45e-5681-457d-b606-d3bab430162c, 5329c45e-5681-457d-b606-d3bab430162c, null","https://staging-tutor.cnx.org/contents/3480b873-4599-4e23-b139-afb532da162b, https://staging-tutor.cnx.org/contents/5329c45e-5681-457d-b606-d3bab430162c, https://staging-tutor.cnx.org/contents/5329c45e-5681-457d-b606-d3bab430162c, null"
+66,cum nobis consectetur,Tutor,Bob Martin,4,"-4, -3, -2, -1","https://exercises-dev.openstax.org/exercise/-4, https://exercises-dev.openstax.org/exercise/-3, https://exercises-dev.openstax.org/exercise/-2, https://exercises-dev.openstax.org/exercise/-1","2017-09-25T21:34:40.085Z, 2017-09-25T21:34:40.096Z, 2017-09-25T21:34:40.107Z, 2017-09-25T21:34:40.117Z",saepe nam mollitia,929ad821-5801-4272-9186-ffb0c047ec08,"1.1, 1.2, 1.2, null","3480b873-4599-4e23-b139-afb532da162b, 5329c45e-5681-457d-b606-d3bab430162c, 5329c45e-5681-457d-b606-d3bab430162c, null","https://staging-tutor.cnx.org/contents/3480b873-4599-4e23-b139-afb532da162b, https://staging-tutor.cnx.org/contents/5329c45e-5681-457d-b606-d3bab430162c, https://staging-tutor.cnx.org/contents/5329c45e-5681-457d-b606-d3bab430162c, null"

spec/lib/openstax/exercises/v1/exercise_spec.rb

@@ -19,7 +19,7 @@
       expect(exercise.preview).to be_nil
       expect(exercise.context).to eq context
       expect(exercise.content).to eq content
-      expect(exercise.url).to eq "#{OpenStax::Exercises::V1.server_url}/exercises/#{hash[:uid]}"
+      expect(exercise.url).to eq "#{OpenStax::Exercises::V1.server_url}/exercise/#{hash[:uid]}"
       expect(exercise.nickname).to eq nickname
       expect(exercise.title).to eq title
       expect(exercise.question_answers[0].length).to eq 2

spec/routines/export_exercise_exclusions_spec.rb

@@ -83,7 +83,7 @@
 
           specify 'with urls' do
             ee_numbers_urls = ee_numbers.map do |number|
-              OpenStax::Exercises::V1.uri_for("/exercises/#{number}").to_s
+              OpenStax::Exercises::V1.uri_for("/exercise/#{number}").to_s
             end
             expect(output[:excluded_exercises_hash].map(&:exercise_url)).to(
               match_array ee_numbers_urls
@@ -123,7 +123,7 @@
           excluded_exercises = @course.excluded_exercises.sort_by(&:exercise_number)
           ee_numbers = excluded_exercises.map(&:exercise_number)
           ee_numbers_urls = ee_numbers.map do |number|
-            OpenStax::Exercises::V1.uri_for("/exercises/#{number}").to_s
+            OpenStax::Exercises::V1.uri_for("/exercise/#{number}").to_s
           end
           exclusion_dates = excluded_exercises.map do |ee|
             DateTimeUtilities.to_api_s(ee.created_at)
@@ -192,7 +192,7 @@
 
         specify 'exercise url' do
           exercise_urls = [@ee_1, @ee_2, @ee_3, @ee_removed].map do |ee|
-            OpenStax::Exercises::V1.uri_for("/exercises/#{ee.exercise_number}").to_s
+            OpenStax::Exercises::V1.uri_for("/exercise/#{ee.exercise_number}").to_s
           end
           outputs_by_exercise.each{ |output| expect(output[:exercise_url]).to be_in exercise_urls }
         end
@@ -232,7 +232,7 @@
         it 'includes all the correct data' do
           exercise_numbers = [@ee_1, @ee_2, @ee_3, @ee_removed].map(&:exercise_number)
           exercise_urls = exercise_numbers.map do |exercise_number|
-            OpenStax::Exercises::V1.uri_for("/exercises/#{exercise_number}").to_s
+            OpenStax::Exercises::V1.uri_for("/exercise/#{exercise_number}").to_s
           end
           pages = [@page_1, @page_2, @page_removed, @exercise_another_eco.page]
           page_uuids = pages.map(&:uuid)