Sugestão: Casos de testes de unidade extras para a função Signer::sign. about sped-common

[robmachado]

Srs. A classe Signer é e deve ser o mais GENÉRICA possível !
Os projetos que fazem uso dessa classe é que devem passar os parâmetros corretos para que a assinatura seja feita corretamente. É dentro desses projetos específicos onde são definidos nos nodes a serem assinados e outros parâmetros construtivos.

[robmachado]

O nivel de cobertura por testes unitários ainda está baixo em vários projetos. Alguns (mais simples) já estão com uma cobertura melhor. Os testes unitários são muito importantes para evitar a QUEBRA de padrões e de funcionalidade. E com eles fica mais difícil acontecerem falhas quando alguém faz uma alteração imprópria.

[lpj145]

na verdade, o interessante, é que cada documento, que implementasse sua ou uma interface Sped tivesse algo como getSignNode ou signedNode coisa do tipo, onde devolveria o nó a ser assinado, podendo ser um ou mais, Sign não deve saber o que esta tentando assinar, ele só deveria pegar essa node, elemento, texto e tentar assinar, no repositorio sign, teríamos um wrapp com strategy pattern, onde TextSign, XmlSign fossem os ideais para essas assinaturas.

[zoiosilva]

Correto, concordo. Acrescento ainda que os testes podem funcionar como uma documentação viva do projeto, testando comportamentos que devem obrigatoriamente ser atendidos para que não quebre projetos dependentes.