Skip to content
GitLab
Explorer
Connexion
S'inscrire
Navigation principale
Rechercher ou aller à…
Projet
S
stats-facil
Gestion
Activité
Membres
Labels
Programmation
Tickets
Tableaux des tickets
Jalons
Wiki
Code
Requêtes de fusion
Dépôt
Branches
Validations
Étiquettes
Graphe du dépôt
Comparer les révisions
Extraits de code
Compilation
Pipelines
Jobs
Planifications de pipeline
Artéfacts
Déploiement
Releases
Registre de paquets
Registre de conteneur
Registre de modèles
Opération
Environnements
Modules Terraform
Surveillance
Incidents
Analyse
Données d'analyse des chaînes de valeur
Analyse des contributeurs
Données d'analyse CI/CD
Données d'analyse du dépôt
Expériences du modèle
Aide
Aide
Support
Documentation de GitLab
Comparer les forfaits GitLab
Forum de la communauté
Contribuer à GitLab
Donner votre avis
Raccourcis clavier
?
Extraits de code
Groupes
Projets
Afficher davantage de fils d'Ariane
facil
stats-facil
Validations
5fe115a5
Valider
5fe115a5
rédigé
9 years ago
par
mattab
Parcourir les fichiers
Options
Téléchargements
Correctifs
Plain Diff
Add integration test for tracking Download and Outlink with fragment
parent
5c73bd13
Aucune branche associée trouvée
Branches contenant la validation
Aucune étiquette associée trouvée
Aucune requête de fusion associée trouvée
Modifications
2
Masquer les modifications d'espaces
En ligne
Côte à côte
Affichage de
2 fichiers modifiés
core/Tracker/Action.php
+3
-2
3 ajouts, 2 suppressions
core/Tracker/Action.php
tests/PHPUnit/Integration/TrackerTest.php
+16
-3
16 ajouts, 3 suppressions
tests/PHPUnit/Integration/TrackerTest.php
avec
19 ajouts
et
5 suppressions
core/Tracker/Action.php
+
3
−
2
Voir le fichier @
5fe115a5
...
@@ -208,8 +208,9 @@ abstract class Action
...
@@ -208,8 +208,9 @@ abstract class Action
protected
function
setActionUrlWithoutExcludingParameters
(
$url
)
protected
function
setActionUrlWithoutExcludingParameters
(
$url
)
{
{
$this
->
rawActionUrl
=
PageUrl
::
getUrlIfLookValid
(
$url
);
$url
=
PageUrl
::
getUrlIfLookValid
(
$url
);
$this
->
actionUrl
=
PageUrl
::
getUrlIfLookValid
(
$url
);
$this
->
rawActionUrl
=
$url
;
$this
->
actionUrl
=
$url
;
}
}
abstract
protected
function
getActionsToLookup
();
abstract
protected
function
getActionsToLookup
();
...
...
Ce diff est replié.
Cliquez pour l'agrandir.
tests/PHPUnit/Integration/TrackerTest.php
+
16
−
3
Voir le fichier @
5fe115a5
...
@@ -59,9 +59,6 @@ class TrackerTest extends IntegrationTestCase
...
@@ -59,9 +59,6 @@ class TrackerTest extends IntegrationTestCase
{
{
parent
::
setUp
();
parent
::
setUp
();
GlobalSettingsProvider
::
unsetSingletonInstance
();
Config
::
unsetInstance
();
Fixture
::
createWebsite
(
'2014-01-01 00:00:00'
);
Fixture
::
createWebsite
(
'2014-01-01 00:00:00'
);
$this
->
tracker
=
new
TestTracker
();
$this
->
tracker
=
new
TestTracker
();
...
@@ -253,6 +250,22 @@ class TrackerTest extends IntegrationTestCase
...
@@ -253,6 +250,22 @@ class TrackerTest extends IntegrationTestCase
$this
->
assertActionEquals
(
'example.com'
,
2
);
$this
->
assertActionEquals
(
'example.com'
,
2
);
}
}
public
function
test_trackRequest_shouldTrackOutlinkWithFragment
()
{
$request
=
$this
->
buildRequest
(
array
(
'idsite'
=>
1
,
'link'
=>
'http://example.com/outlink#fragment-here'
,
'rec'
=>
1
));
$this
->
tracker
->
trackRequest
(
$request
);
$this
->
assertActionEquals
(
'http://example.com/outlink#fragment-here'
,
1
);
}
public
function
test_trackRequest_shouldTrackDownloadWithFragment
()
{
$request
=
$this
->
buildRequest
(
array
(
'idsite'
=>
1
,
'download'
=>
'http://example.com/file.zip#fragment-here&pk_campaign=Campaign param accepted here'
,
'rec'
=>
1
));
$this
->
tracker
->
trackRequest
(
$request
);
$this
->
assertActionEquals
(
'http://example.com/file.zip#fragment-here&pk_campaign=Campaign param accepted here'
,
1
);
}
public
function
test_main_shouldReturnEmptyPiwikResponse_IfNoRequestsAreGiven
()
public
function
test_main_shouldReturnEmptyPiwikResponse_IfNoRequestsAreGiven
()
{
{
$requestSet
=
$this
->
getEmptyRequestSet
();
$requestSet
=
$this
->
getEmptyRequestSet
();
...
...
Ce diff est replié.
Cliquez pour l'agrandir.
Aperçu
0%
Chargement en cours
Veuillez réessayer
ou
joindre un nouveau fichier
.
Annuler
You are about to add
0
people
to the discussion. Proceed with caution.
Terminez d'abord l'édition de ce message.
Enregistrer le commentaire
Annuler
Veuillez vous
inscrire
ou vous
se connecter
pour commenter