mumuki-laboratory 8.6.1 → 9.0.4

data/lib/mumuki/laboratory/version.rb

@@ -1,5 +1,5 @@
 module Mumuki
   module Laboratory
-    VERSION = '8.6.1'
+    VERSION = '9.0.4'
   end
 end

data/spec/controllers/certificates_controller_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe CertificatesController, type: :controller, organization_workspace: :test do
+  let(:user) { create(:user) }
+  let(:certificate) { create(:certificate, user: user) }
+
+  context 'download' do
+    before { set_current_user! user }
+    before { get :download, params: { code: certificate.code } }
+
+    it { expect(response.status).to eq 200 }
+    it { expect(response.body).to_not be_nil }
+    it { expect(response.content_type).to eq 'application/pdf' }
+  end
+end

data/spec/controllers/organizations_api_controller_spec.rb

@@ -34,7 +34,7 @@ describe Api::OrganizationsController, type: :controller, organization_workspace
 
         context 'with wildcard permissions' do
           before { get :index }
-          let(:api_client) { create :api_client, role: :janitor, grant: '*' }
+          let(:api_client) { create :api_client, role: :admin, grant: '*' }
 
           it { check_status! 200 }
 
@@ -42,7 +42,7 @@ describe Api::OrganizationsController, type: :controller, organization_workspace
         end
         context 'with non-wildcard permissions' do
           before { get :index }
-          let(:api_client) { create :api_client, role: :janitor, grant: 'public/*:private/*' }
+          let(:api_client) { create :api_client, role: :admin, grant: 'public/*:private/*' }
 
           it { check_status! 200 }
 
@@ -54,14 +54,14 @@ describe Api::OrganizationsController, type: :controller, organization_workspace
         context 'with a public organization' do
           before { get :show, params: {id: 'public'} }
 
-          context 'with a user without janitor permissions' do
+          context 'with a user without admin permissions' do
             let(:api_client) { create :api_client, role: :editor, grant: 'another_organization/*' }
 
             it { check_status! 403 }
           end
 
-          context 'with a user with janitor' do
-            let(:api_client) { create :api_client, role: :janitor, grant: 'public/*' }
+          context 'with a user with admin' do
+            let(:api_client) { create :api_client, role: :admin, grant: 'public/*' }
 
             it { check_status! 200 }
           end
@@ -70,7 +70,7 @@ describe Api::OrganizationsController, type: :controller, organization_workspace
         context 'with an organization that has a dot in its name' do
           before { get :show, params: {id: 'dot.org'} }
 
-          let(:api_client) { create :api_client, role: :janitor, grant: 'dot.org/*' }
+          let(:api_client) { create :api_client, role: :admin, grant: 'dot.org/*' }
           it { check_status! 200 }
         end
 
@@ -84,7 +84,7 @@ describe Api::OrganizationsController, type: :controller, organization_workspace
           end
 
           context 'with a user with permissions' do
-            let(:api_client) { create :api_client, role: :janitor, grant: 'private/*' }
+            let(:api_client) { create :api_client, role: :admin, grant: 'private/*' }
 
             it { check_status! 200 }
             it { expect(response.body).to json_like(private_organization.to_resource_h) }
@@ -181,7 +181,7 @@ describe Api::OrganizationsController, type: :controller, organization_workspace
       end
 
 
-      context 'with not-janitor permissions' do
+      context 'with non-admin permissions' do
         let(:api_client) { create :api_client, role: :editor, grant: '*' }
 
         it { check_status! 403 }
@@ -228,7 +228,14 @@ describe Api::OrganizationsController, type: :controller, organization_workspace
         it { expect(updated_organizaton.contact_email).to eq 'second_email@gmail.com' }
       end
 
-      context 'with not-janitor permissions' do
+      context 'with janitor permissions' do
+        let(:api_client) { create :api_client, role: :janitor, grant: 'existing-organization/*' }
+        before { put :update, params: {id: 'existing-organization', organization: update_json} }
+
+        it { check_status! 403 }
+      end
+
+      context 'with non-admin permissions' do
         let(:api_client) { create :api_client, role: :teacher, grant: 'existing-organization/*' }
         before { put :update, params: {id: 'existing-organization', organization: update_json} }
 

data/spec/dummy/db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20210119190204) do
+ActiveRecord::Schema.define(version: 20210302181654) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -71,6 +71,29 @@ ActiveRecord::Schema.define(version: 20210119190204) do
     t.index ["slug"], name: "index_books_on_slug", unique: true
   end
 
+  create_table "certificate_programs", force: :cascade do |t|
+    t.string "title"
+    t.string "template_html_erb"
+    t.text "description"
+    t.string "background_image_url"
+    t.bigint "organization_id"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["organization_id"], name: "index_certificate_programs_on_organization_id"
+  end
+
+  create_table "certificates", force: :cascade do |t|
+    t.bigint "user_id"
+    t.bigint "certificate_program_id"
+    t.datetime "start_date"
+    t.datetime "end_date"
+    t.string "code"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["certificate_program_id"], name: "index_certificates_on_certificate_program_id"
+    t.index ["user_id"], name: "index_certificates_on_user_id"
+  end
+
   create_table "chapters", id: :serial, force: :cascade do |t|
     t.datetime "created_at"
     t.datetime "updated_at"
@@ -377,6 +400,7 @@ ActiveRecord::Schema.define(version: 20210119190204) do
     t.boolean "wins_page"
     t.boolean "immersible"
     t.boolean "prevent_manual_evaluation_content"
+    t.text "faqs"
     t.index ["book_id"], name: "index_organizations_on_book_id"
     t.index ["name"], name: "index_organizations_on_name", unique: true
   end

data/spec/features/certificate_programs_flow_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+feature 'Certificate Programs flow', organization_workspace: :test do
+
+  let(:user) { create :user, first_name: 'Foo', last_name: 'Bar' }
+
+  before { set_current_user! user }
+
+  before { Organization.find_by_name('test').switch! }
+  before { create :certificate, user: user, code: 'abc' }
+
+  before { visit '/certificates/verify/abc' }
+
+  scenario { expect(page).to have_text('Foo Bar') }
+  scenario { expect(page).to have_link(href: /linkedin\.com\/profile\/add/) }
+  scenario { expect(page).to have_link(href: /certificates\/download\/abc/) }
+end

data/spec/features/exercise_flow_spec.rb

@@ -106,19 +106,19 @@ feature 'Exercise Flow', organization_workspace: :test do
       scenario 'visit exercise by id, standalone mode' do
         visit "/exercises/#{problem_1.id}"
         expect(page).to have_text('Functional Programming 1')
-        expect(page).to have_text('Profile')
+        expect(page).to have_text('My account')
       end
       scenario 'visit exercise by id, embedded mode in non embeddable organization' do
         visit "/exercises/#{problem_1.id}?embed=true"
         expect(page).to have_text('Functional Programming 1')
-        expect(page).to have_text('Profile')
+        expect(page).to have_text('My account')
       end
       scenario 'visit exercise by id, embedded mode in embeddable organization' do
         Organization.current.tap { |it| it.embeddable = true }.save!
 
         visit "/exercises/#{problem_1.id}?embed=true"
         expect(page).to_not have_text('Functional Programming 1')
-        expect(page).to_not have_text('Profile')
+        expect(page).to_not have_text('My account')
       end
     end
 

data/spec/features/login_flow_spec.rb

@@ -64,6 +64,6 @@ feature 'Login Flow', organization_workspace: :test do
 
     expect(page).to_not have_text('Sign in')
     expect(page).to have_text('Sign Out')
-    expect(page).to have_text('Profile')
+    expect(page).to have_text('My profile')
   end
 end

data/spec/features/menu_bar_spec.rb

@@ -16,9 +16,10 @@ feature 'menu bar' do
       scenario 'should not see menu bar' do
         visit '/'
 
-        expect(page).not_to have_text('Profile')
+        expect(page).not_to have_text('My account')
         expect(page).not_to have_text('Classroom')
         expect(page).not_to have_text('Bibliotheca')
+        expect(page).not_to have_text('FAQs')
       end
     end
 
@@ -30,11 +31,12 @@ feature 'menu bar' do
       scenario 'should not see menu bar' do
         visit '/'
 
-        expect(page).not_to have_text('Profile')
+        expect(page).not_to have_text('My account')
         expect(page).not_to have_text('Classroom')
         expect(page).not_to have_text('Bibliotheca')
         expect(page).not_to have_text('Solve other\'s doubts')
         expect(page).not_to have_text('My doubts')
+        expect(page).not_to have_text('FAQs')
       end
     end
   end
@@ -48,55 +50,69 @@ feature 'menu bar' do
     let(:admin) { create(:user, permissions: {student: 'private/*', admin: 'private/*'}) }
     let(:owner) { create(:user, permissions: {student: 'private/*', owner: 'private/*'}) }
 
-    scenario 'visitor should only see profile' do
+    scenario 'visitor should only see their account' do
       set_current_user! visitor
 
       visit '/'
-      expect(page).to have_text('Profile')
+      expect(page).to have_text('My account')
       expect(page).not_to have_text('Classroom')
       expect(page).not_to have_text('Bibliotheca')
       expect(page).not_to have_text('Solve other\'s doubts')
       expect(page).not_to have_text('My doubts')
+      expect(page).not_to have_text('FAQs')
     end
 
     context 'student with no discussions should' do
-      scenario 'only see profile if forum is not enabled' do
+      scenario 'only see their account if forum is not enabled' do
         set_current_user! student
 
         visit '/'
-        expect(page).to have_text('Profile')
+        expect(page).to have_text('My account')
         expect(page).not_to have_text('Classroom')
         expect(page).not_to have_text('Bibliotheca')
         expect(page).not_to have_text('Solve other\'s doubts')
         expect(page).not_to have_text('My doubts')
+        expect(page).not_to have_text('FAQs')
       end
 
-      scenario 'see profile and solve_other_doubts links if forum is enabled' do
+      scenario 'see their account and solve_other_doubts links if forum is enabled' do
         set_current_user! student
         private_organization.update! forum_enabled: true
 
         visit '/'
-        expect(page).to have_text('Profile')
+        expect(page).to have_text('My account')
         expect(page).not_to have_text('Classroom')
         expect(page).not_to have_text('Bibliotheca')
         expect(page).to have_text('Solve other\'s doubts')
         expect(page).not_to have_text('My doubts')
+        expect(page).not_to have_text('FAQs')
+      end
+    end
+
+    context 'organization with faqs' do
+      before { Organization.current.update! faqs: "Some faqs" }
+
+      scenario 'should see FAQs link' do
+        visit '/'
+
+        expect(page).to have_text('FAQs')
       end
     end
 
     context 'student with discussions should' do
       let(:discussion) { create(:discussion, item: lesson.exercises.last, initiator: student)}
 
-      scenario 'only see profile if forum is not enabled' do
+      scenario 'only see their account if forum is not enabled' do
         set_current_user! student
         student.subscribe_to! discussion
 
         visit '/'
-        expect(page).to have_text('Profile')
+        expect(page).to have_text('My account')
         expect(page).not_to have_text('Classroom')
         expect(page).not_to have_text('Bibliotheca')
         expect(page).not_to have_text('Solve other\'s doubts')
         expect(page).not_to have_text('My doubts')
+        expect(page).not_to have_text('FAQs')
       end
 
       scenario 'see all discussions links if forum is enabled' do
@@ -105,25 +121,27 @@ feature 'menu bar' do
         student.subscribe_to! discussion
 
         visit '/'
-        expect(page).to have_text('Profile')
+        expect(page).to have_text('My account')
         expect(page).not_to have_text('Classroom')
         expect(page).not_to have_text('Bibliotheca')
         expect(page).to have_text('Solve other\'s doubts')
         expect(page).to have_text('My doubts')
+        expect(page).not_to have_text('FAQs')
       end
 
-      scenario 'only see profile if forum is enabled in a forum_only_for_trusted organization' do
+      scenario 'only see their account if forum is enabled in a forum_only_for_trusted organization' do
         set_current_user! student
         student.subscribe_to! discussion
         private_organization.update! forum_enabled: true
         private_organization.update! forum_only_for_trusted: true
 
         visit '/'
-        expect(page).to have_text('Profile')
+        expect(page).to have_text('My account')
         expect(page).not_to have_text('Classroom')
         expect(page).not_to have_text('Bibliotheca')
         expect(page).not_to have_text('Solve other\'s doubts')
         expect(page).not_to have_text('My doubts')
+        expect(page).not_to have_text('FAQs')
       end
 
       scenario 'see all discussions links if forum is enabled in a forum_only_for_trusted organization but it is trusted' do
@@ -134,62 +152,64 @@ feature 'menu bar' do
         private_organization.update! forum_only_for_trusted: true
 
         visit '/'
-        expect(page).to have_text('Profile')
+        expect(page).to have_text('My account')
         expect(page).not_to have_text('Classroom')
         expect(page).not_to have_text('Bibliotheca')
         expect(page).to have_text('Solve other\'s doubts')
         expect(page).to have_text('My doubts')
+        expect(page).not_to have_text('FAQs')
       end
     end
 
-    scenario 'teacher should see profile and classroom' do
+    scenario 'teacher should see their account and classroom' do
       set_current_user! teacher
 
       visit '/'
 
-      expect(page).to have_text('Profile')
+      expect(page).to have_text('My account')
       expect(page).to have_text('Classroom')
       expect(page).not_to have_text('Bibliotheca')
       expect(page).not_to have_text('Solve other\'s doubts')
       expect(page).not_to have_text('My doubts')
+      expect(page).not_to have_text('FAQs')
     end
 
-    scenario 'writer should see profile and bibliotheca' do
+    scenario 'writer should see their account and bibliotheca' do
       set_current_user! writer
 
       visit '/'
 
-      expect(page).to have_text('Profile')
+      expect(page).to have_text('My account')
       expect(page).not_to have_text('Classroom')
       expect(page).to have_text('Bibliotheca')
     end
 
-    scenario 'janitor should see profile and classroom' do
+    scenario 'janitor should see their account and classroom' do
       set_current_user! janitor
 
       visit '/'
 
-      expect(page).to have_text('Profile')
+      expect(page).to have_text('My account')
       expect(page).to have_text('Classroom')
       expect(page).not_to have_text('Bibliotheca')
     end
 
-    scenario 'admin should see profile, classroom and bibliotheca' do
+    scenario 'admin should their account, classroom and bibliotheca' do
       set_current_user! admin
 
       visit '/'
 
-      expect(page).to have_text('Profile')
+      expect(page).to have_text('My account')
       expect(page).to have_text('Classroom')
       expect(page).to have_text('Bibliotheca')
     end
 
-    scenario 'owner should see profile, classroom and bibliotheca' do
+    scenario 'owner should see their account, classroom and bibliotheca' do
       set_current_user! owner
 
       visit '/'
 
-      expect(page).to have_text('Profile')
+      expect(page).to have_text('My account')
       expect(page).to have_text('Classroom')
       expect(page).to have_text('Bibliotheca')
     end

data/spec/features/profile_flow_spec.rb

@@ -97,31 +97,41 @@ feature 'Profile Flow', organization_workspace: :test do
 
           click_on(button_options)
           expect(page).to have_text('Your data was updated successfully')
-          expect(page).to have_text('Profile')
+          expect(page).to have_text('My profile')
         end
       end
     end
 
     context 'with no messages' do
-
-      scenario 'visit messages tab' do
-        visit "/user#messages"
+      scenario 'visit messages' do
+        visit "/user/messages"
 
         expect(page).to have_text('It seems you don\'t have any messages yet!')
       end
     end
 
     context 'with messages' do
-      scenario 'visit messages tab' do
+      scenario 'visit messages' do
         Organization.find_by_name('test').switch!
         problem.submit_solution! user, {content: 'something'}
         Message.import_from_resource_h! message
-        visit "/user#messages"
+        visit "/user/messages"
 
         expect(page).to_not have_text('It seems you don\'t have any messages yet!')
         expect(page).to have_text(problem.name)
       end
     end
-  end
 
+    context 'visit certificates tab' do
+      before { Organization.find_by_name('test').switch! }
+      before { create :certificate, user: user, code: 'abc' }
+      before { visit '/user/certificates' }
+
+      scenario { expect(page).to have_text('Test - Certificate program to test') }
+      scenario { expect(page).to have_link(href: /certificates\/verify\/abc/) }
+      scenario { expect(page).to have_link(href: /linkedin.com\/profile\/add/) }
+      scenario { expect(page).to have_link(href: /certificates\/download\/abc/) }
+    end
+
+  end
 end