Update snapshots to fix tests

[edelarua]

Pull Request

Fixes #1092

[github-actions]

Unit Tests Summary

  1 files   33 suites   2s ⏱️
150 tests 150 ✅   0 💤 0 ❌
282 runs  170 ✅ 112 💤 0 ❌

Results for commit 0a4fef8.

♻️ This comment has been updated with latest results.

[shajoezhu]

hi @edelarua , I was wondering, if you could change the snapshot a little, just make it fail. I was wondering if the snapshot test is actually ran by the cicd pipeline.

[edelarua]

hi @edelarua , I was wondering, if you could change the snapshot a little, just make it fail. I was wondering if the snapshot test is actually ran by the cicd pipeline.

@shajoezhu I believe the snapshot tests aren't run for PRs since these checks never fail until tested in the integration tests

[shajoezhu]

hi @edelarua , I was wondering, if you could change the snapshot a little, just make it fail. I was wondering if the snapshot test is actually ran by the cicd pipeline.

@shajoezhu I believe the snapshot tests aren't run for PRs since these checks never fail until tested in the integration tests

thanks a lot @edelarua for confirming. I had also followed up with @cicdguy before, and these will be resolve in coming weeks, we will need these snapshot test ran on gh cicd

[cicdguy]

Indeed. We'll have both CRAN as well as non-CRAN runs enabled on all repos soon.

[ayogasekaram]

lgtm! thanks @edelarua