Skip to content

Added an api test for content group fields#41147

Merged
MacJoom merged 5 commits intojoomla:4.4-devfrom
rajputanuj31:content_field-group
Jul 22, 2023
Merged

Added an api test for content group fields#41147
MacJoom merged 5 commits intojoomla:4.4-devfrom
rajputanuj31:content_field-group

Conversation

@rajputanuj31
Copy link
Contributor

@rajputanuj31 rajputanuj31 commented Jul 12, 2023

Added an api test for content group fields.

@brianteeman
Copy link
Contributor

brianteeman commented Jul 13, 2023

something has gone wrong here.

The pr and the code say "content group fields."
But the file name is tests/System/integration/api/com_contact/Field_Group.cy.js

@rajputanuj31
Copy link
Contributor Author

rajputanuj31 commented Jul 15, 2023

something has gone wrong here.

The pr and the code say "content group fields." But the file name is tests/System/integration/api/com_contact/Field_Group.cy.js

Fixed.

@rajputanuj31
Copy link
Contributor Author

rajputanuj31 commented Jul 15, 2023

something has gone wrong here.

The pr and the code say "content group fields." But the file name is tests/System/integration/api/com_contact/Field_Group.cy.js

POST API test is failing, I am unable to find the reason
I will be grateful if you help me with this.

@MacJoom
Copy link
Contributor

MacJoom commented Jul 17, 2023

May be related to #40610 ?

@rajputanuj31
Copy link
Contributor Author

rajputanuj31 commented Jul 18, 2023 •
edited
Loading

May be related to #40610 ?

It's working. After successful merge of #40610 this PR can be merged.
thanks a lot.

@rajputanuj31
Copy link
Contributor Author

rajputanuj31 commented Jul 21, 2023 •
edited
Loading

@MacJoom Is PR #40610 merged in 4.4-dev??

@MacJoom
Copy link
Contributor

MacJoom commented Jul 21, 2023

Yes it should be in the upmerge. But I see that drone is still failing so I need to check

@MacJoom
Copy link
Contributor

MacJoom commented Jul 21, 2023

No - it wasn't ... i will do another upmerge

@rajputanuj31
Copy link
Contributor Author

rajputanuj31 commented Jul 21, 2023

No - it wasn't ... i will do another upmerge

Ok

@MacJoom MacJoom merged commit 217b967 into joomla:4.4-dev Jul 22, 2023
@MacJoom MacJoom added this to the Joomla! 4.4.0 milestone Jul 22, 2023
GeraintEdwards pushed a commit to GeraintEdwards/joomla-cms that referenced this pull request Aug 14, 2023
@rajputanuj31 @Geraint2
Added an api test for content group fields (joomla#41147)
6336a62 
* added an api test for content group fields

* changing file location

---------
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

PR-4.4-dev Unit/System Tests

Projects

None yet

Milestone

Joomla! 4.4.0

Development

Successfully merging this pull request may close these issues.

4 participants

@rajputanuj31 @brianteeman @MacJoom @joomla-cms-bot