8000 feat: abrir URLs em lote com TDD — ciclos 9 e 10 by Duda-RM · Pull Request #3 · xLucasMelo/linkding · GitHub
[go: up one dir, main page]

Skip to content

feat: abrir URLs em lote com TDD — ciclos 9 e 10 - #3

Open
Duda-RM wants to merge 9 commits into
masterfrom
tdd-cycles-9-10
Open

Duda-RM wants to merge 9 commits into
masterfrom
tdd-cycles-9-10

Conversation

@Duda-RM
@Duda-RM Duda-RM commented Nov 1, 2026
Copy link
Copy Markdown
Collaborator

Substitui o PR #2 após a renomeação da branch para tdd-cycles-9-10. Os commits permanecem os mesmos.

A issue sissbruecker#1345 já tinha cópia de URLs na interface, mas não oferecia a abertura em lote. Esta contribuição adiciona “Open URLs (this page)”: um clique em Execute abre os favoritos marcados da página atual, na ordem de exibição, sem enviar o formulário ou alterar a área de transferência.

A abertura acontece diretamente no manipulador do clique com _blank e noopener,noreferrer. A seleção entre páginas fica desmarcada e oculta para essa ação; o rótulo informa o escopo. As ações existentes de copiar e excluir mantêm os comportamentos verificados pelos testes.

Foram executados dois ciclos reais de TDD, com commits distintos para RED, GREEN e REFACTOR de cada ciclo:

  • Ciclo 9: abrir um favorito selecionado e expor a ação no menu.
  • Ciclo 10: abrir vários favoritos na ordem da página, ignorar itens desmarcados/sem link e tratar seleção vazia.

Validação: 5 testes unitários do componente JavaScript e 111 testes Django dos arquivos de ações, listagem principal e arquivados aprovados. Compilação npm run build-js, Prettier e Ruff passaram. Logs, horários, hashes e quadros de resultados estão em docs/ptoss2/tdd-cycles-9-10/.

Os testes JavaScript simulam os limites do navegador. Não houve validação E2E ou manual da abertura real de abas; o navegador pode bloquear pop-ups. Os 111 testes Django não representam a suíte completa do projeto.

Referência: sissbruecker#1345

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.

1 participant

0