Skip to content

Commit

Permalink
chore: phpcs e definição fixa de tomeout para testes automatizados co…
Browse files Browse the repository at this point in the history 
…m rest (#779)

* chore: phpcs e definição fixa de tomeout para testes automatizados

* chore: atualizar analise phpcs para php-8.2
  • Loading branch information
@mrglaydson
mrglaydson authored Jan 21, 2025
1 parent 391648a commit 4b5dc21
Show file tree
Hide file tree
Showing 12 changed files with 54 additions and 61 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/phpcs.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: '7.3'
php-version: '8.2'
coverage: none
tools: composer

Expand Down
2 changes: 1 addition & 1 deletion phpcs.xml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
<config name="installed_paths"
value="vendor/pheromone/phpcs-security-audit,vendor/phpcompatibility/php-compatibility" />
<config name="minimum_supported_wp_version" value="4.4.0" />
<config name="testVersion" value="5.6-" />
<config name="testVersion" value="8.2-" />

<file>src</file>
<file>tests</file>
Expand Down
67 changes: 31 additions & 36 deletions src/rn/ProcessoEletronicoRN.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -142,11 +142,6 @@ protected function inicializarObjInfraIBanco()
return BancoSEI::getInstance();
}


/**
* Construtor do objeto SoapClien utilizado para comunicação Webservice
*
*/
private function getObjPenWs()
{
if (InfraString::isBolVazia($this->strEnderecoWebService)) {
Expand Down Expand Up @@ -1536,9 +1531,9 @@ private function montarDadosMaisDeUmComponenteDigital($objDocumento, $parStrNume

foreach ($arrayComponentesDigitais as $indice => $objComponenteDigital){

if (is_array($objComponenteDigital)) {
$objComponenteDigital = (object) $objComponenteDigital;
}
if (is_array($objComponenteDigital)) {
$objComponenteDigital = (object) $objComponenteDigital;
}

$contComponentes++;
$objComponenteDigitalDTO = new ComponenteDigitalDTO();
Expand Down Expand Up @@ -1568,18 +1563,18 @@ private function montarDadosMaisDeUmComponenteDigital($objDocumento, $parStrNume
$objComponenteDigitalDTO->setStrDadosComplementares($objComponenteDigital->dadosComplementaresDoTipoDeArquivo);

//Registrar componente digital necessita ser enviado pelo trâmite específico //TODO: Teste $parObjComponentesDigitaisSolicitados aqui
if(isset($parObjComponentesDigitaisSolicitados)) {
$arrObjItensSolicitados = isset($parObjComponentesDigitaisSolicitados) ? $parObjComponentesDigitaisSolicitados : array($parObjComponentesDigitaisSolicitados);
foreach ($arrObjItensSolicitados as $objItemSolicitado) {
if(!is_null($objItemSolicitado)){
$objItemSolicitado['hashes'] = is_array($objItemSolicitado['hashes']) ? $objItemSolicitado['hashes'] : array($objItemSolicitado['hashes']);
if(isset($parObjComponentesDigitaisSolicitados)) {
$arrObjItensSolicitados = isset($parObjComponentesDigitaisSolicitados) ? $parObjComponentesDigitaisSolicitados : array($parObjComponentesDigitaisSolicitados);
foreach ($arrObjItensSolicitados as $objItemSolicitado) {
if(!is_null($objItemSolicitado)){
$objItemSolicitado['hashes'] = is_array($objItemSolicitado['hashes']) ? $objItemSolicitado['hashes'] : array($objItemSolicitado['hashes']);

if($objItemSolicitado['protocolo'] == $objComponenteDigitalDTO->getStrProtocolo() && in_array($strHashConteudo, $objItemSolicitado['hashes']) && !$objDocumento->retirado) {
$objComponenteDigitalDTO->setStrSinEnviar("S");
}
if($objItemSolicitado['protocolo'] == $objComponenteDigitalDTO->getStrProtocolo() && in_array($strHashConteudo, $objItemSolicitado['hashes']) && !$objDocumento->retirado) {
$objComponenteDigitalDTO->setStrSinEnviar("S");
}
}
}
}

//TODO: Avaliar dados do tamanho do documento em bytes salvo na base de dados
$objComponenteDigitalDTO->setNumTamanho($objComponenteDigital->tamanhoEmBytes);
Expand Down Expand Up @@ -2323,28 +2318,28 @@ public static function desbloquearProcesso($parDblIdProcedimento)

public static function comparacaoOrdemAjustadaDocumentos($parDocumento1, $parDocumento2)
{
if (is_array($parDocumento1)){
$numOrdemDocumento1 = isset($parDocumento1['ordemAjustada']) ? intval($parDocumento1['ordemAjustada']) : intval($parDocumento1['ordem']);
$numOrdemDocumento2 = isset($parDocumento2['ordemAjustada']) ? intval($parDocumento2['ordemAjustada']) : intval($parDocumento2['ordem']);
}
else{
$numOrdemDocumento1 = isset($parDocumento1->ordemAjustada) ? intval($parDocumento1->ordemAjustada) : intval($parDocumento1->ordem);
$numOrdemDocumento2 = isset($parDocumento2->ordemAjustada) ? intval($parDocumento2->ordemAjustada) : intval($parDocumento2->ordem);
}
if (is_array($parDocumento1)){
$numOrdemDocumento1 = isset($parDocumento1['ordemAjustada']) ? intval($parDocumento1['ordemAjustada']) : intval($parDocumento1['ordem']);
$numOrdemDocumento2 = isset($parDocumento2['ordemAjustada']) ? intval($parDocumento2['ordemAjustada']) : intval($parDocumento2['ordem']);
}
else{
$numOrdemDocumento1 = isset($parDocumento1->ordemAjustada) ? intval($parDocumento1->ordemAjustada) : intval($parDocumento1->ordem);
$numOrdemDocumento2 = isset($parDocumento2->ordemAjustada) ? intval($parDocumento2->ordemAjustada) : intval($parDocumento2->ordem);
}

return $numOrdemDocumento1 - $numOrdemDocumento2;
}

public static function comparacaoOrdemDocumentos($parDocumento1, $parDocumento2)
{

if (is_array($parDocumento1)) {
$parDocumento1 = (object) $parDocumento1;
}
if (is_array($parDocumento1)) {
$parDocumento1 = (object) $parDocumento1;
}

if (is_array($parDocumento2)) {
$parDocumento2 = (object) $parDocumento2;
}
if (is_array($parDocumento2)) {
$parDocumento2 = (object) $parDocumento2;
}

$numOrdemDocumento1 = intval($parDocumento1->ordem);
$numOrdemDocumento2 = intval($parDocumento2->ordem);
Expand All @@ -2353,13 +2348,13 @@ public static function comparacaoOrdemDocumentos($parDocumento1, $parDocumento2)

public static function comparacaoOrdemComponenteDigitais($parComponenteDigital1, $parComponenteDigital2)
{
if (is_array($parComponenteDigital1)) {
$parComponenteDigital1 = (object) $parComponenteDigital1;
}
if (is_array($parComponenteDigital1)) {
$parComponenteDigital1 = (object) $parComponenteDigital1;
}

if (is_array($parComponenteDigital2)) {
$parComponenteDigital2 = (object) $parComponenteDigital2;
}
if (is_array($parComponenteDigital2)) {
$parComponenteDigital2 = (object) $parComponenteDigital2;
}

$numOrdemComponenteDigital1 = intval($parComponenteDigital1->ordem);
$numOrdemComponenteDigital2 = intval($parComponenteDigital2->ordem);
Expand Down
16 changes: 8 additions & 8 deletions src/rn/ReceberProcedimentoRN.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2844,16 +2844,16 @@ private function criarDiretorioAnexo($parObjAnexoDTO)
private function adicionarObservacoesSobreNumeroDocumento($parObjDocumento)
{
$arrObjObservacoes = array();
if (is_array($parObjDocumento->produtor)) {
$parObjDocumento->produtor = (object) $parObjDocumento->produtor;
}
if (is_array($parObjDocumento->produtor)) {
$parObjDocumento->produtor = (object) $parObjDocumento->produtor;
}

$strNumeroDocumentoOrigem = isset($parObjDocumento->protocolo) ? $parObjDocumento->protocolo : $parObjDocumento->produtor->numeroDeIdentificacao;
if(!empty($strNumeroDocumentoOrigem)){
$objObservacaoDTO = new ObservacaoDTO();
$objObservacaoDTO->setStrDescricao("Número do Documento na Origem: " . $strNumeroDocumentoOrigem);
$arrObjObservacoes[] = $objObservacaoDTO;
}
if(!empty($strNumeroDocumentoOrigem)){
$objObservacaoDTO = new ObservacaoDTO();
$objObservacaoDTO->setStrDescricao("Número do Documento na Origem: " . $strNumeroDocumentoOrigem);
$arrObjObservacoes[] = $objObservacaoDTO;
}

return $arrObjObservacoes;
}
Expand Down
2 changes: 2 additions & 0 deletions tests_sei41/funcional/tests/CenarioBaseTestCase.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -471,6 +471,8 @@ protected function validarHistoricoTramite(
}
}



protected function validarDadosProcesso($descricao, $restricao, $observacoes, $listaInteressados, $hipoteseLegal = null)
{
sleep(2);
Expand Down
7 changes: 3 additions & 4 deletions tests_sei41/funcional/tests/RecebimentoRecusaJustificativaGrandeTest.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,7 @@ public function setUp(): void

// Carregar contexto de testes e dados sobre certificado digital
$this->destinatarioWs = $this->definirContextoTeste(CONTEXTO_ORGAO_B);

// Instanciar objeto de teste utilizando o BeSimpleSoap

$localCertificado = $this->destinatarioWs['LOCALIZACAO_CERTIFICADO_DIGITAL'];
$senhaCertificado = $this->destinatarioWs['SENHA_CERTIFICADO_DIGITAL'];
$this->servicoPEN = $this->instanciarApiDeIntegracao($localCertificado, $senhaCertificado);
Expand Down Expand Up @@ -99,7 +98,7 @@ private function recusarTramite($id_tramite)
$parametros = new stdClass();
$parametros->recusaDeTramite = new stdClass();
$parametros->recusaDeTramite->IDT = $id_tramite;
$parametros->recusaDeTramite->justificativa = utf8_encode($justificativa);
$parametros->recusaDeTramite->justificativa = mb_convert_encoding($justificativa, 'UTF-8', 'ISO-8859-1');
$parametros->recusaDeTramite->motivo = "99";

return $this->recusarTramiteAPI($parametros);
Expand All @@ -117,7 +116,7 @@ private function instanciarApiDeIntegracao($localCertificado, $senhaCertificado)

$strClientGuzzle = new GuzzleHttp\Client([
'base_uri' => $strBaseUri,
'timeout' => 5.0,
'timeout' => ProcessoEletronicoRN::WS_TIMEOUT_CONEXAO,
'headers' => $arrheaders,
'cert' => [$localCertificado, $senhaCertificado],
]);
Expand Down
1 change: 0 additions & 1 deletion tests_sei41/funcional/tests/TramiteProcessoComDocumentoRestritoTest.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ public function test_tramitar_processo_com_documento_restrito()
self::$remetente['LOGIN'],
self::$remetente['SENHA']
);


self::$protocoloTeste = $this->cadastrarProcessoFixture(self::$processoTeste); // Cadastrar novo processo de teste
self::$documentoTeste["RESTRICAO"] = \ProtocoloRN::$NA_RESTRITO; // Configuração de documento restrito
Expand Down
2 changes: 1 addition & 1 deletion tests_sei41/funcional/tests/TramiteRecebimentoDocumentoAnexadoTest.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ private function instanciarApiDeIntegracao($localCertificado, $senhaCertificado)

$strClientGuzzle = new GuzzleHttp\Client([
'base_uri' => PEN_ENDERECO_WEBSERVICE,
'timeout' => 40,
'timeout' => ProcessoEletronicoRN::WS_TIMEOUT_CONEXAO,
'headers' => $arrheaders,
'cert' => [$localCertificado, $senhaCertificado],
]);
Expand Down
2 changes: 1 addition & 1 deletion tests_sei41/funcional/tests/TramiteRecebimentoDocumentoAvulsoTest.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -358,7 +358,7 @@ private function instanciarApiDeIntegracao($localCertificado, $senhaCertificado)

$strClientGuzzle = new GuzzleHttp\Client([
'base_uri' => PEN_ENDERECO_WEBSERVICE,
'timeout' => 5.0,
'timeout' => ProcessoEletronicoRN::WS_TIMEOUT_CONEXAO,
'headers' => $arrheaders,
'cert' => [$localCertificado, $senhaCertificado],
]);
Expand Down
5 changes: 2 additions & 3 deletions tests_sei41/funcional/tests/TramiteRecebimentoInteressadosDuplicadosTest.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ public function test_envio_processo_com_documento_anexado()
// Atribui dois interessados utilizando o mesmo nome
self::$processoTeste['INTERESSADOS'] = array("Interessado com mesmo nome", "Interessado com mesmo nome");

// Instanciar objeto de teste utilizando o BeSimpleSoap
// Instanciar objeto de teste utilizando o Guzzle
$localCertificado = self::$remetente['LOCALIZACAO_CERTIFICADO_DIGITAL'];
$senhaCertificado = self::$remetente['SENHA_CERTIFICADO_DIGITAL'];
self::$servicoPEN = $this->instanciarApiDeIntegracao($localCertificado, $senhaCertificado);
Expand Down Expand Up @@ -76,8 +76,7 @@ private function instanciarApiDeIntegracao($localCertificado, $senhaCertificado)

$strClientGuzzle = new GuzzleHttp\Client([
'base_uri' => PEN_ENDERECO_WEBSERVICE,
'handler' => GuzzleHttp\HandlerStack::create(),
'timeout' => 5.0,
'timeout' => ProcessoEletronicoRN::WS_TIMEOUT_CONEXAO,
'headers' => $arrheaders,
'cert' => [$localCertificado, $senhaCertificado],
]);
Expand Down
5 changes: 2 additions & 3 deletions tests_sei41/funcional/tests/TramiteRecebimentoMultiplosComponentesDigitais.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ public function setUp(): void
self::$contextoOrgaoA = $this->definirContextoTeste(CONTEXTO_ORGAO_A);
self::$contextoOrgaoB = $this->definirContextoTeste(CONTEXTO_ORGAO_B);

// Instanciar objeto de teste utilizando o BeSimpleSoap
// Instanciar objeto de teste utilizando o Guzzle
$localCertificado = self::$contextoOrgaoB['LOCALIZACAO_CERTIFICADO_DIGITAL'];
$senhaCertificado = self::$contextoOrgaoB['SENHA_CERTIFICADO_DIGITAL'];
self::$servicoPEN = $this->instanciarApiDeIntegracao($localCertificado, $senhaCertificado);
Expand Down Expand Up @@ -211,8 +211,7 @@ private function instanciarApiDeIntegracao($localCertificado, $senhaCertificado)

$strClientGuzzle = new GuzzleHttp\Client([
'base_uri' => PEN_ENDERECO_WEBSERVICE,
'handler' => GuzzleHttp\HandlerStack::create(),
'timeout' => 40,
'timeout' => ProcessoEletronicoRN::WS_TIMEOUT_CONEXAO,
'headers' => $arrheaders,
'cert' => [$localCertificado, $senhaCertificado],
]);
Expand Down
4 changes: 2 additions & 2 deletions ...s_sei41/funcional/tests/TramiteRecebimentoMultiplosComponentesDigitaisApenasPendentes.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ public function setUp(): void
self::$contextoOrgaoA = $this->definirContextoTeste(CONTEXTO_ORGAO_A);
self::$contextoOrgaoB = $this->definirContextoTeste(CONTEXTO_ORGAO_B);

// Instanciar objeto de teste utilizando o BeSimpleSoap
// Instanciar objeto de teste utilizando o Guzzle
$localCertificado = self::$contextoOrgaoA['LOCALIZACAO_CERTIFICADO_DIGITAL'];
$senhaCertificado = self::$contextoOrgaoA['SENHA_CERTIFICADO_DIGITAL'];
self::$servicoPEN = $this->instanciarApiDeIntegracao($localCertificado, $senhaCertificado);
Expand Down Expand Up @@ -179,7 +179,7 @@ private function instanciarApiDeIntegracao($localCertificado, $senhaCertificado)

$strClientGuzzle = new GuzzleHttp\Client([
'base_uri' => PEN_ENDERECO_WEBSERVICE,
'timeout' => 40,
'timeout' => ProcessoEletronicoRN::WS_TIMEOUT_CONEXAO,
'headers' => $arrheaders,
'cert' => [$localCertificado, $senhaCertificado],
]);
Expand Down

0 comments on commit 4b5dc21

Please sign in to comment.