diff --git a/Gemfile b/Gemfile index d034287e..e33df326 100644 --- a/Gemfile +++ b/Gemfile @@ -8,7 +8,7 @@ gem 'rails', '~> 5.2.3' # Use postgresql as the database for Active Record gem 'pg', '>= 0.18', '< 2.0' # Use Puma as the app server -gem 'puma', '~> 3.11' +gem 'puma', '~> 3.12' # Use SCSS for stylesheets gem 'sass-rails', '~> 5.0' # Use Uglifier as compressor for JavaScript assets @@ -64,6 +64,8 @@ group :test do # database_cleaner is not required, but highly recommended gem 'database_cleaner' gem 'shoulda-matchers' + gem 'cucumber-rails-training-wheels' + gem 'launchy' end # Windows does not include zoneinfo files, so bundle the tzinfo-data gem diff --git a/Gemfile.lock b/Gemfile.lock index 1c0f8f1e..b6223e46 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -90,6 +90,8 @@ GEM mime-types (>= 2.0, < 4) nokogiri (~> 1.8) railties (>= 4.2, < 7) + cucumber-rails-training-wheels (1.0.0) + cucumber-rails (>= 1.1.1) cucumber-tag_expressions (1.1.1) cucumber-wire (0.0.1) database_cleaner (1.7.0) @@ -110,6 +112,8 @@ GEM concurrent-ruby (~> 1.0) jbuilder (2.9.1) activesupport (>= 4.2.0) + launchy (2.5.0) + addressable (~> 2.7) listen (3.1.5) rb-fsevent (~> 0.9, >= 0.9.4) rb-inotify (~> 0.9, >= 0.9.7) @@ -133,13 +137,13 @@ GEM multi_json (1.14.1) multi_test (0.1.2) nio4r (2.5.2) - nokogiri (1.10.5) + nokogiri (1.10.8) mini_portile2 (~> 2.4.0) orm_adapter (0.5.0) pg (1.1.4) public_suffix (4.0.1) - puma (3.12.1) - rack (2.0.7) + puma (3.12.6) + rack (2.2.3) rack-test (1.1.0) rack (>= 1.0, < 3) rails (5.2.3) @@ -243,7 +247,7 @@ GEM selenium-webdriver (>= 3.0, < 4.0) websocket-driver (0.7.1) websocket-extensions (>= 0.1.0) - websocket-extensions (0.1.4) + websocket-extensions (0.1.5) xpath (3.2.0) nokogiri (~> 1.8) @@ -256,12 +260,14 @@ DEPENDENCIES capybara (>= 2.15) coffee-rails (~> 4.2) cucumber-rails + cucumber-rails-training-wheels database_cleaner devise jbuilder (~> 2.5) + launchy listen (>= 3.0.5, < 3.2) pg (>= 0.18, < 2.0) - puma (~> 3.11) + puma (~> 3.12) rails (~> 5.2.3) rspec-rails sass-rails (~> 5.0) diff --git a/features/adicionar_comentario_wiki.feature b/features/adicionar_comentario_wiki.feature new file mode 100644 index 00000000..b35b7aab --- /dev/null +++ b/features/adicionar_comentario_wiki.feature @@ -0,0 +1,26 @@ + +#language: pt +#encoding: utf-8 + +Funcionalidade: Adicionar comentário a um processo de atendimento + Como um secretário, + para que eu possa adicionar informações a um processo de atendimento, + sem alterar o conteúdo original, + eu gostaria de adicionar um comentário nesse processo de atendimento. + + Contexto: + Dado que eu esteja conectado como usuario "alicinha@gatinha.com", "123456", "secretario" + E que esteja na página "processos de atendimento" + E eu seleciono um processo + E eu clico no botão "Adicionar comentário" + + Cenário: Preenche campo de comentário + Quando eu preencho o campo "comentário" com "Processo acompanhado também pela professora Maristela Holanda" + E clico no botão "Confirmar" + Então sou redirecionado para a página "processos de atendimento" + E recebo uma mensagem de sucesso + + Cenário: Deixa o campo de comentário vazio + Quando eu não preencho o campo "comentário" + E clico no botão "Confirmar" + Então recebo uma mensagem de erro \ No newline at end of file diff --git a/features/anexar_novo_processo.feature b/features/anexar_novo_processo.feature new file mode 100644 index 00000000..a52826ce --- /dev/null +++ b/features/anexar_novo_processo.feature @@ -0,0 +1,23 @@ +#language:pt + +Funcionalidade: Anexar arquivo a um processo existente + Como um secretário, + para que eu possa detalhar mais os processos da Wiki, + eu gostaria de anexar um ou mais documentos em um processo já existente + + Contexto: + Dado que eu esteja conectado como usuario "usuario_secretario@secretario.com", "123456", "secretario" + E que esteja na página "processos de atendimento" + E clico no botão "Adicionar processo" + + Cenário: Anexar um arquivo válido + Quando eu clico no botão "Adicionar documento" + E envio o arquivo "documento.pdf" + E clico no botão "Confirmar" + Então sou redirecionado para a página "processos de atendimento" + E recebo uma mensagem de sucesso + + Cenário: Anexar arquivo nválido + Quando eu clico no botão "Adicionar documento" + E envio o arquivo "Pabllo_Vittar_seu_crime.mp4" + Então recebo uma mensagem de erro \ No newline at end of file diff --git a/features/anexar_processo.feature b/features/anexar_processo.feature new file mode 100644 index 00000000..2b55a654 --- /dev/null +++ b/features/anexar_processo.feature @@ -0,0 +1,24 @@ +#language:pt + +Funcionalidade: Anexar arquivo a um processo existente + Como um secretário, + para que eu possa detalhar mais os processos da Wiki, + eu gostaria de anexar um ou mais documentos em um processo já existente + + Contexto: + Dado que eu esteja conectado como usuario "alicinha@gatinha.com", "123456", "secretario" + E que esteja na página "processos de atendimentoo" + E seleciono um processo + E clico no botão "Editar processo" + + Cenário: Anexar um arquivo válido + Quando eu clico no botão "Adicionar documento" + E envio o arquivo "documento.pdf" + E clico no botão "Confirmar" + Então sou redirecionado para a página "processos de atendimentoo" + E recebo uma mensagem de sucesso + + Cenário: Anexar arquivo nválido + Quando eu clico no botão "Adicionar documento" + E envio o arquivo "Pabllo_Vittar_seu_crime.mp4" + Então recebo uma mensagem de erro \ No newline at end of file diff --git a/features/consultar_processo.feature b/features/consultar_processo.feature new file mode 100644 index 00000000..8ab80278 --- /dev/null +++ b/features/consultar_processo.feature @@ -0,0 +1,21 @@ +#language: pt +Funcionalidade: Consultar processo de atendimento na Wiki + Como um secretário, + Quero consultar uma entrada na Wiki sobre um processo de atendimento, + Para que eu possa me informar sobre como um processo é realizado + + Contexto: + Dado que eu esteja conectado como usuario "usuario_secretario@secretario.com", "123456", "secretario" + E que esteja na página "processos de atendimento" + E seleciono um processo + E eu clicar no botão "Ver detalhes" + + Cenário: Consultar um processo de atendimento existente + Então sou redirecionado para a página "informações do processo" + E devo poder visualizar o "número SEI" + E devo poder visualizar a "responsável" + E devo poder visualizar o "status" + E devo poder visualizar os "documentos" + + Cenário: Consultar um processo de atendimento inexistente + Então recebo uma mensagem de erro diff --git a/features/criar_processo.feature b/features/criar_processo.feature new file mode 100644 index 00000000..d9f3a780 --- /dev/null +++ b/features/criar_processo.feature @@ -0,0 +1,25 @@ +#language: pt + +Funcionalidade: Cadastrar um novo processo na Wiki de atendimento + + COMO um secretario + EU QUERO cadastrar um novo processo na Wiki de atendimento + A FIM de disponibilizar informações para outros secretários. + + Contexto: + Dado que eu esteja conectado como usuario "usuario_secretario@secretario.com", "123456", "secretario" + E que esteja na página "processos de atendimento" + E clico no botão "Adicionar processo" + + Cenário: Preenche campos com informações válidas(caminho feliz) + Quando eu preencho o campo "número SEI" com "0423-123456/2017" + E preencho o campo "reponsável" com "Genaína Rodrigues" + E seleciono o "status" "em andamento" + E clico no botão "Confirmar" + Então sou redirecionado para a página "processos de atendimento" + E recebo uma mensagem de sucesso + + Cenário: Preenche campos com informações inválidas + Quando eu preencho o campo "número SEI" com "alilicece" + E clico no botão "Confirmar" + Então recebo uma mensagem de erro \ No newline at end of file diff --git a/features/editar_processo_wiki.feature b/features/editar_processo_wiki.feature new file mode 100644 index 00000000..c5b75b92 --- /dev/null +++ b/features/editar_processo_wiki.feature @@ -0,0 +1,27 @@ + +#language: pt +#encoding: utf-8 + +Funcionalidade: Editar processo de atendimento + Como um secretário cadastrado, + para que eu possa manter a Wiki atualizada, + eu gostaria de editar um processo de atendimento já cadastrado + + Contexto: + Dado que eu esteja conectado como usuario "alicinha@gatinha.com", "123456", "secretario" + E que esteja na página "processos de atendimento" + E eu seleciono um processo + E eu clico no botão "Editar processo" + + Cenário: Preenche campos com informações válidas + Quando eu preencho o campo "número SEI" com "0423-123456/2017" + E preencho o campo "reponsável" com "Genaína Rodrigues" + E seleciono o "status" "em andamento" + E clico no botão "Confirmar" + Então sou redirecionado para a página "processos de atendimento" + E recebo uma mensagem de sucesso + + Cenário: Preenche campos com informações inválidas + Quando eu preencho o campo "número SEI" com "alilicece" + E clico no botão "Confirmar" + Então recebo uma mensagem de erro \ No newline at end of file diff --git a/features/step_definitions/processos_steps.rb b/features/step_definitions/processos_steps.rb new file mode 100644 index 00000000..29a5d769 --- /dev/null +++ b/features/step_definitions/processos_steps.rb @@ -0,0 +1,6 @@ +Dado(/seleciono um processo/) do + pending # Write code here that turns the phrase above into concrete actions +end + + + \ No newline at end of file diff --git a/features/step_definitions/shared_steps.rb b/features/step_definitions/shared_steps.rb new file mode 100644 index 00000000..5a01237a --- /dev/null +++ b/features/step_definitions/shared_steps.rb @@ -0,0 +1,36 @@ +Quando(/(?:clico|clicar) no botão "([^"]*)"/) do |botao| + pending # Write code here that turns the phrase above into concrete actions +end + +Então(/recebo uma mensagem de (sucesso|erro)/) do |status| + pending # Write code here that turns the phrase above into concrete actions +end + +Quando(/não preencho o campo "([^"]*)"/) do |campo| + pending # Write code here that turns the phrase above into concrete actions +end + +Quando(/preencho o campo "([^"]*)" com "([^"]*)"/) do |campo, valor| + pending # Write code here that turns the phrase above into concrete actions +end + +Dado(/(?:esteja na|redirecionado para a) página "([^"]*)"/) do |pagina| + pending # Write code here that turns the phrase above into concrete actions +end + +Dado(/esteja conectado como usuario "([^"]*)", "([^"]*)", "([^"]*)"/) do |usuario, senha, funcao| + pending # Write code here that turns the phrase above into concrete actions +end + +Quando(/visualizar (?:o|a|os|as) "([^"]*)"/) do |elemento| + pending +end + +Quando(/seleciono (?:o|a|os|as) "([^"]*)" "([^"]*)"/) do |campo, valor| + pending +end + +Quando(/envio o arquivo "([^"]*)"/) do |arquivo| + pending # Write code here that turns the phrase above into concrete actions +end + \ No newline at end of file diff --git a/features/step_definitions/web_steps.rb b/features/step_definitions/web_steps.rb new file mode 100644 index 00000000..4d9aab64 --- /dev/null +++ b/features/step_definitions/web_steps.rb @@ -0,0 +1,254 @@ +# TL;DR: YOU SHOULD DELETE THIS FILE +# +# This file was generated by Cucumber-Rails and is only here to get you a head start +# These step definitions are thin wrappers around the Capybara/Webrat API that lets you +# visit pages, interact with widgets and make assertions about page content. +# +# If you use these step definitions as basis for your features you will quickly end up +# with features that are: +# +# * Hard to maintain +# * Verbose to read +# +# A much better approach is to write your own higher level step definitions, following +# the advice in the following blog posts: +# +# * http://benmabey.com/2008/05/19/imperative-vs-declarative-scenarios-in-user-stories.html +# * http://dannorth.net/2011/01/31/whose-domain-is-it-anyway/ +# * http://elabs.se/blog/15-you-re-cuking-it-wrong +# + + +require 'uri' +require 'cgi' +require File.expand_path(File.join(File.dirname(__FILE__), "..", "support", "paths")) +require File.expand_path(File.join(File.dirname(__FILE__), "..", "support", "selectors")) + +module WithinHelpers + def with_scope(locator) + locator ? within(*selector_for(locator)) { yield } : yield + end +end +World(WithinHelpers) + +# Single-line step scoper +When /^(.*) within (.*[^:])$/ do |step, parent| + with_scope(parent) { When step } +end + +# Multi-line step scoper +When /^(.*) within (.*[^:]):$/ do |step, parent, table_or_string| + with_scope(parent) { When "#{step}:", table_or_string } +end + +Given /^(?:|I )am on (.+)$/ do |page_name| + visit path_to(page_name) +end + +When /^(?:|I )go to (.+)$/ do |page_name| + visit path_to(page_name) +end + +When /^(?:|I )press "([^"]*)"$/ do |button| + click_button(button) +end + +When /^(?:|I )follow "([^"]*)"$/ do |link| + click_link(link) +end + +When /^(?:|I )fill in "([^"]*)" with "([^"]*)"$/ do |field, value| + fill_in(field, :with => value) +end + +When /^(?:|I )fill in "([^"]*)" for "([^"]*)"$/ do |value, field| + fill_in(field, :with => value) +end + +# Use this to fill in an entire form with data from a table. Example: +# +# When I fill in the following: +# | Account Number | 5002 | +# | Expiry date | 2009-11-01 | +# | Note | Nice guy | +# | Wants Email? | | +# +# TODO: Add support for checkbox, select or option +# based on naming conventions. +# +When /^(?:|I )fill in the following:$/ do |fields| + fields.rows_hash.each do |name, value| + When %{I fill in "#{name}" with "#{value}"} + end +end + +When /^(?:|I )select "([^"]*)" from "([^"]*)"$/ do |value, field| + select(value, :from => field) +end + +When /^(?:|I )check "([^"]*)"$/ do |field| + check(field) +end + +When /^(?:|I )uncheck "([^"]*)"$/ do |field| + uncheck(field) +end + +When /^(?:|I )choose "([^"]*)"$/ do |field| + choose(field) +end + +When /^(?:|I )attach the file "([^"]*)" to "([^"]*)"$/ do |path, field| + attach_file(field, File.expand_path(path)) +end + +Then /^(?:|I )should see "([^"]*)"$/ do |text| + if page.respond_to? :should + page.should have_content(text) + else + assert page.has_content?(text) + end +end + +Then /^(?:|I )should see \/([^\/]*)\/$/ do |regexp| + regexp = Regexp.new(regexp) + + if page.respond_to? :should + page.should have_xpath('//*', :text => regexp) + else + assert page.has_xpath?('//*', :text => regexp) + end +end + +Then /^(?:|I )should not see "([^"]*)"$/ do |text| + if page.respond_to? :should + page.should have_no_content(text) + else + assert page.has_no_content?(text) + end +end + +Then /^(?:|I )should not see \/([^\/]*)\/$/ do |regexp| + regexp = Regexp.new(regexp) + + if page.respond_to? :should + page.should have_no_xpath('//*', :text => regexp) + else + assert page.has_no_xpath?('//*', :text => regexp) + end +end + +Then /^the "([^"]*)" field(?: within (.*))? should contain "([^"]*)"$/ do |field, parent, value| + with_scope(parent) do + field = find_field(field) + field_value = (field.tag_name == 'textarea') ? field.text : field.value + if field_value.respond_to? :should + field_value.should =~ /#{value}/ + else + assert_match(/#{value}/, field_value) + end + end +end + +Then /^the "([^"]*)" field(?: within (.*))? should not contain "([^"]*)"$/ do |field, parent, value| + with_scope(parent) do + field = find_field(field) + field_value = (field.tag_name == 'textarea') ? field.text : field.value + if field_value.respond_to? :should_not + field_value.should_not =~ /#{value}/ + else + assert_no_match(/#{value}/, field_value) + end + end +end + +Then /^the "([^"]*)" field should have the error "([^"]*)"$/ do |field, error_message| + element = find_field(field) + classes = element.find(:xpath, '..')[:class].split(' ') + + form_for_input = element.find(:xpath, 'ancestor::form[1]') + using_formtastic = form_for_input[:class].include?('formtastic') + error_class = using_formtastic ? 'error' : 'field_with_errors' + + if classes.respond_to? :should + classes.should include(error_class) + else + assert classes.include?(error_class) + end + + if page.respond_to?(:should) + if using_formtastic + error_paragraph = element.find(:xpath, '../*[@class="inline-errors"][1]') + error_paragraph.should have_content(error_message) + else + page.should have_content("#{field.titlecase} #{error_message}") + end + else + if using_formtastic + error_paragraph = element.find(:xpath, '../*[@class="inline-errors"][1]') + assert error_paragraph.has_content?(error_message) + else + assert page.has_content?("#{field.titlecase} #{error_message}") + end + end +end + +Then /^the "([^"]*)" field should have no error$/ do |field| + element = find_field(field) + classes = element.find(:xpath, '..')[:class].split(' ') + if classes.respond_to? :should + classes.should_not include('field_with_errors') + classes.should_not include('error') + else + assert !classes.include?('field_with_errors') + assert !classes.include?('error') + end +end + +Then /^the "([^"]*)" checkbox(?: within (.*))? should be checked$/ do |label, parent| + with_scope(parent) do + field_checked = find_field(label)['checked'] + if field_checked.respond_to? :should + field_checked.should be_true + else + assert field_checked + end + end +end + +Then /^the "([^"]*)" checkbox(?: within (.*))? should not be checked$/ do |label, parent| + with_scope(parent) do + field_checked = find_field(label)['checked'] + if field_checked.respond_to? :should + field_checked.should be_false + else + assert !field_checked + end + end +end + +Then /^(?:|I )should be on (.+)$/ do |page_name| + current_path = URI.parse(current_url).path + if current_path.respond_to? :should + current_path.should == path_to(page_name) + else + assert_equal path_to(page_name), current_path + end +end + +Then /^(?:|I )should have the following query string:$/ do |expected_pairs| + query = URI.parse(current_url).query + actual_params = query ? CGI.parse(query) : {} + expected_params = {} + expected_pairs.rows_hash.each_pair{|k,v| expected_params[k] = v.split(',')} + + if actual_params.respond_to? :should + actual_params.should == expected_params + else + assert_equal expected_params, actual_params + end +end + +Then /^show me the page$/ do + save_and_open_page +end diff --git a/features/support/paths.rb b/features/support/paths.rb new file mode 100644 index 00000000..290543c3 --- /dev/null +++ b/features/support/paths.rb @@ -0,0 +1,38 @@ +# TL;DR: YOU SHOULD DELETE THIS FILE +# +# This file is used by web_steps.rb, which you should also delete +# +# You have been warned +module NavigationHelpers + # Maps a name to a path. Used by the + # + # When /^I go to (.+)$/ do |page_name| + # + # step definition in web_steps.rb + # + def path_to(page_name) + case page_name + + when /^the home\s?page$/ + '/' + + # Add more mappings here. + # Here is an example that pulls values out of the Regexp: + # + # when /^(.*)'s profile page$/i + # user_profile_path(User.find_by_login($1)) + + else + begin + page_name =~ /^the (.*) page$/ + path_components = $1.split(/\s+/) + self.send(path_components.push('path').join('_').to_sym) + rescue NoMethodError, ArgumentError + raise "Can't find mapping from \"#{page_name}\" to a path.\n" + + "Now, go and add a mapping in #{__FILE__}" + end + end + end +end + +World(NavigationHelpers) diff --git a/features/support/selectors.rb b/features/support/selectors.rb new file mode 100644 index 00000000..33bebc1d --- /dev/null +++ b/features/support/selectors.rb @@ -0,0 +1,44 @@ +# TL;DR: YOU SHOULD DELETE THIS FILE +# +# This file is used by web_steps.rb, which you should also delete +# +# You have been warned +module HtmlSelectorsHelpers + # Maps a name to a selector. Used primarily by the + # + # When /^(.+) within (.+)$/ do |step, scope| + # + # step definitions in web_steps.rb + # + def selector_for(locator) + case locator + + when "the page" + "html > body" + + # Add more mappings here. + # Here is an example that pulls values out of the Regexp: + # + # when /^the (notice|error|info) flash$/ + # ".flash.#{$1}" + + # You can also return an array to use a different selector + # type, like: + # + # when /the header/ + # [:xpath, "//header"] + + # This allows you to provide a quoted selector as the scope + # for "within" steps as was previously the default for the + # web steps: + when /^"(.+)"$/ + $1 + + else + raise "Can't find mapping from \"#{locator}\" to a selector.\n" + + "Now, go and add a mapping in #{__FILE__}" + end + end +end + +World(HtmlSelectorsHelpers) diff --git a/spec/controllers/processos_controller_spec.rb b/spec/controllers/processos_controller_spec.rb new file mode 100644 index 00000000..8bf3d14e --- /dev/null +++ b/spec/controllers/processos_controller_spec.rb @@ -0,0 +1,53 @@ +require 'rails_helper' +require 'capybara/rails' + +RSpec.describe ProcessosController, type: :controller do + describe "#create" do + it 'calls the create method from the Processos model' + describe "valid params" do + it 'saves the process in DB' + it "responds with an success HTTP status" + it 'redirects to process index page' + end + describe "invalid params" do + it "doesn't save the process in DB" + it "responds with an error HTTP status" + end + end + + describe "#update" do + it 'calls the update method from an instance of the Processos model' + describe "valid params" do + it 'updates the process data in the DB' + it "responds with an success HTTP status" + it 'redirects to process index page' + end + describe "invalid params" do + it "doesn't update the process data in the DB" + it "responds with an error HTTP status" + end + end + + describe "#show" do + describe "valid process" do + it "responds with an success HTTP status" + it 'redirects to process show page' + end + describe "invalid process" do + it "responds with an error HTTP status" + end + end + + describe "add comment" do + it "calls the create method from the Comentarios model" + describe "valid params" do + it 'saves the comment in DB' + it "responds with an success HTTP status" + it 'redirects to comment index page' + end + describe "invalid params" do + it "doesn't save the comment in DB" + it "responds with an error HTTP status" + end + end +end diff --git a/spec/models/comentarios_spec.rb b/spec/models/comentarios_spec.rb new file mode 100644 index 00000000..45169498 --- /dev/null +++ b/spec/models/comentarios_spec.rb @@ -0,0 +1,9 @@ +require 'rails_helper' + +RSpec.describe Comentario, type: :model do + describe "#create" do + it "should create a process comment when the attribute are valid" + it "should not create a process comment when the content is blank" + it "should not create a content when the conten id too large" + end +end diff --git a/spec/models/processos_spec.rb b/spec/models/processos_spec.rb new file mode 100644 index 00000000..26976d75 --- /dev/null +++ b/spec/models/processos_spec.rb @@ -0,0 +1,19 @@ +require 'rails_helper' + +RSpec.describe Processos, type: :model do + describe "#create" do + it 'should create a new process when the attributes are valid' + it 'should not create a new process when the sei_process_code is not a valid code' + it 'should not create a new process when there is no assigned user' + it 'should not create a new process when the process_status_id is invalid' + it 'should not create a new process when trying to attach an invalid file ' + end + + describe "#update" do + it 'should update the process when the attributes are valid' + it 'should not update the process when the sei_process_code is not a valid code' + it 'should not update the process when there is no assigned user' + it 'should not update the process when the process_status_id is invalid' + it 'should not update the process when trying to attach an invalid file' + end +end