Skip to content

feat(page-job-scheduler): permite configuração de frequência - #1102

Merged
alinelariguet merged 1 commit into
masterfrom
job-scheduler/DTHFUI-5526
Nov 21, 2021
Merged

feat(page-job-scheduler): permite configuração de frequência#1102
alinelariguet merged 1 commit into
masterfrom
job-scheduler/DTHFUI-5526

Conversation

@alinelariguet

@alinelariguet alinelariguet commented Nov 19, 2021

Copy link
Copy Markdown
Member

PAGE JOB SCHEDULER

DTHFUI-5526


PR Checklist [Revisor]

  • Padrão de Commit (Coeso, de acordo com o que está sendo realizado)
  • Código (Boas práticas, nome de variavéis/métodos, etc.)
  • Testes unitários (Cobre a situação implementada e coverage está mantido)
  • Documentação (Clara, objetiva e com exemplos caso necessário)
  • Samples (A implementação possui exemplo no Labs/Caso de uso)
  • Rodado em navegadores suportados (Chrome, FireFox, Edge)

Qual o comportamento atual?
Não é possível configurar a frequência

Qual o novo comportamento?
Permite configurar a frequência

Simulação
Simular através do sample do portal.
app 2.zip
PR no po-sample-api - WIP

@jcorrea97

Copy link
Copy Markdown
Contributor

Ao selecionar a periodicidade mensal e não selecionar a frequência(primeira etapa de agendamento), está retornando "undefined" na informação da execução(ultima etapa de conclusão).

image

@alinelariguet
alinelariguet force-pushed the job-scheduler/DTHFUI-5526 branch from 4cf02e3 to a80ee07 Compare November 19, 2021 16:01

spyOn(component['poPageJobSchedulerService'], 'convertToJobSchedulerInternal').and.returnValue(returnValue);
component.value = jobSchedulerInternal;
// expect(component.model instanceof PoPageJobSchedulerInternal).toBeTruthy();

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

código comentado

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

corrigido.

expect(result).toBe(expectedValue);
});

it('getFrequencyValue: should return `frequency[value] - frequency[type]`', () => {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

descrição do teste igual ao anterior

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

corrigido.

expect(value).toEqual(valueExpected);
});

it('removeInvalidKeys: should remove invalid keys if keys is `rangeExecutions` and periodicity is `single`', () => {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

descrição do teste igual ao anterior

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

corrigido.

Permite configuração da frequência no componente PoPageJobScheduler.

Fixes DTHFUI-5526
@alinelariguet
alinelariguet force-pushed the job-scheduler/DTHFUI-5526 branch from 6f46ab5 to 33e1153 Compare November 19, 2021 20:19
@alinelariguet
alinelariguet merged commit de052f7 into master Nov 21, 2021
@alinelariguet
alinelariguet deleted the job-scheduler/DTHFUI-5526 branch November 21, 2021 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants