fix(blocks): Rollback to change position value only at validation

[codecov[bot]]

Codecov Report

Merging #494 into develop will decrease coverage by 0.01%. The diff coverage is 96.92%.

@@            Coverage Diff             @@
##           develop    #494      +/-   ##
==========================================
- Coverage    93.22%   93.2%   -0.02%     
==========================================
  Files          229     229              
  Lines         6933    6950      +17     
==========================================
+ Hits          6463    6478      +15     
- Misses         470     472       +2

Impacted Files	Coverage Δ
spec/models/block_spec.rb	100% <100%> (ø)	:arrow_up:
spec/support/blueprints.rb	98.3% <100%> (+0.01%)	:arrow_up:
...ontrollers/mobilizations/blocks_controller_spec.rb	100% <100%> (ø)	:arrow_up:
app/policies/block_policy.rb	100% <100%> (ø)	:arrow_up:
spec/factories/mobilization.rb	98.91% <100%> (+0.01%)	:arrow_up:
config/routes.rb	100% <100%> (ø)	:arrow_up:
app/models/block.rb	96.15% <87.5%> (-3.85%)	:arrow_down:
app/controllers/mobilizations/blocks_controller.rb	97.43% <93.33%> (-2.57%)	:arrow_down:

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1edc051...b8c52ca. Read the comment docs.

[hudsilva]

Atualizações realizadas:

Método create POST /block Foi necessário atualizar o mesmo pois ao criar um novo bloco ele estourava erro caso não fosse um bloco válido. Obs: O front deverá que passar a position de um bloco novo ao cria-lo. (Será necessário devido a remoção do controle de posições no model Block (no after_save)

• Removido before_validation : set_position and after_save :switch_position do model Block
• Adicionado update_blocks: esse método recebe uma lista de id - position e atualiza os respectivos blocos
• adicionado PUT /blocks nova rota para atualização das posições dos blocos
• Atualização da policy block. Para que seja possível autorizar o acesso ao método batch_update via controller.

Extras: Aproveitei e corrigi alguns testes e factories utilizadas nessa parte.

@igr-santos A rota para atualizar a lista ficou

PUT /mobilizations/:mobilization_id/blocks

• Esse método funciona para 2 ou mais blocos passados. Quando for passado apenas um ele irá retornar uma mensagem avisando que é necessário 2 ou mais.

params:

{
    "mobilization_id": 1,
    "blocks": [{
            "id": 20,
            "position": 2
        },
        {
            "id": 21,
            "position": 1
        }
    ]
}

[hudsilva]

@lpirola acho legal você dar um check também nas mudanças :)