47 Commits

Author SHA1 Message Date
Chad A
7a0f850fd5 Create add_to_project.yml 2025-04-15 09:46:09 -07:00
mjtalbot
4561b9d112 chore: flush rive runtime change
Diffs=
59d5d4d658 chore: flush rive runtime change (#9428)
f2968887ec Add silvers without lfs (#9425)
eabc82a4c2 chore: temporarily remove silvers (#9424)
6cce6709cb refactor(vulkan): Lift lifecycle management (#9410)
7f5e51f4b4 refactor(renderer): Send map sizes to unmap functions as well (#9407)
43d6ac25e6 Metal fiddle context assert fix (#9393)

Co-authored-by: Chris Dalton <99840794+csmartdalton@users.noreply.github.com>
Co-authored-by: Jonathon Copeland <jcopela4@gmail.com>
Co-authored-by: Maxwell Talbot <talbot.maxwell@gmail.com>
2025-04-11 16:28:35 +00:00
philter
6e162b26f9 rive-runtime pull silver rivs
# Description
rive-runtime is building, but now its failing on the silvers. I **think** it could be because we need to git pull lfs the srivs so trying that.

Diffs=
9d516a9ece rive-runtime pull silver rivs (#9384)

Co-authored-by: Philip Chung <philterdesign@gmail.com>
2025-04-04 22:14:21 +00:00
philter
df94fae694 Fix glfw working path
# Description
Previous PR to fix rive-runtime test had the path relative to mono, not relative to rive-runtime.

Diffs=
bd512ec1d9 Fix glfw working path (#9382)

Co-authored-by: Philip Chung <philterdesign@gmail.com>
2025-04-04 20:52:08 +00:00
philter
dc26d62e99 Fix Rive-runtime tests
# Description
Mac unit tests for the downstream rive-runtime have been failing for some time. Looks like glfw wasn't being built in the downstream workflow.

# Details
This PR adds the Build glfw step that we do in our mono repo into the rive-runtime test workflow. Ran it locally, was able to repro the test error and this fixes the issue locally.

```
../../../../renderer/path_fiddle/fiddle_context_gl.cpp:23:10: fatal error: 'GLFW/glfw3.h' file not found
hb-ot-map.cc
#include "GLFW/glfw3.h"
         ^~~~~~~~~~~~~~
```

Diffs=
92a4d0a287 Fix Rive-runtime tests (#9381)

Co-authored-by: Philip Chung <philterdesign@gmail.com>
2025-04-04 20:05:53 +00:00
mjtalbot
4d8dffb2e1 bump premake
Diffs=
bc6010b6e bump premake (#8120)

Co-authored-by: Maxwell Talbot <talbot.maxwell@gmail.com>
2024-09-11 09:02:32 +00:00
rivessamr
5207dd8ba2 Opensource (tools) tests as part of runtime
- Most of tools/* have been moved to the new packages/runtime/tests. (Excludes fuzz and a few things with proprietary information)

- The tools/test are now in packages/runtime/unit_tests/renderer.

- The runtime/tests are now also moved, to the following packages/runtime/unit_tests/runtime (moved the "dev/tests" to packages/runtime/unit_tests)

- Thus unit_tests comprises of runtime and renderer tests (in the future we can add more categories as folder, this avoids too many tests in one folder)

- The rendering tests and runtime tests are built together in one executable called unit_tests*. This is what is described in the rive-runtime README - it is the way our developers can test the rendering (GPU) and runtime (CPU) unit tests in one go.

- pr_runtime_tests now builds and runs all the unit_tests, previously it was only executing the runtime tests, but now it includes the renderer tests for the (same platforms still tested: windows, windows msvc, mac, linux)

- The pr_tools_tests still tests the same workloads on the same devices, limited still to the rendering tests

ISSUES:

- We cannot build all the "tools" tests (e.g. imagediff, rendering tests) with the runtime tests. The runtime tests enables TESTING. I'm also not sure if running the runtime tests on all the devices is actually is worth it - if the tests are platform+device agnostic then probably not worth it.

- Windows pr_runtime_tests switched to ws-actions/configure-aws-credentials@v1 from microsoft/setup-msbuild@v1.1, as would not build otherwise, seems related to old version of premake

TODO:

- Verify of the performance is reasonable for runner / checkins

Diffs=
f25ee97a0 Opensource (tools) tests as part of runtime (#8035)

Co-authored-by: rivessamr <suki@rive.app>
2024-09-03 22:45:48 +00:00
umberto-sonnino
8812887efe Fix Android goldens
It also removes premake5 alpha and bump a few deps

Diffs=
5c14a4a30 Fix Android goldens (#7902)

Co-authored-by: Umberto Sonnino <umberto@rive.app>
2024-08-22 07:51:55 +00:00
luigi-rosso
ab10ebec06 Fix downstream cpp tests
Still trying! Logging more details now.

Diffs=
1cf6a65f1 Fix downstream cpp tests (#6643)

Co-authored-by: Luigi Rosso <luigi-rosso@users.noreply.github.com>
2024-02-16 19:23:47 +00:00
luigi-rosso
b1d54ea0ec Single test script for windows and mac.
Diffs=
a35883508 Single test script for windows and mac. (#6642)

Co-authored-by: Luigi Rosso <luigi-rosso@users.noreply.github.com>
2024-02-16 18:33:30 +00:00
mjtalbot
c9a52af321 Pushing merge, resolved using upstream.\n\n message=update mono to keep details in commit and not pr
gonna double check it and merge it...

previous job failed on hte pr message, probably too complex

https://github.com/rive-app/rive/actions/runs/7929209375/job/21648989390

```
+ gh pr create --fill --title 'There have been merge conflicts' --body 'diff --cc .github/workflows/tests.yml
index 76659125,e20119eb..00000000
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@@ -58,8 -58,9 +58,12 @@@ jobs
          run: |
            cd dev/test
            C:\premake5.exe --scripts=..\..\build vs2022
++<<<<<<< ours
++=======
+           cd out/debug
++>>>>>>> theirs
            MSBuild rive.sln
-           build\bin\debug\tests.exe
+           tests.exe

    build-macos:
      runs-on: macOS-latest'
pull request create failed: GraphQL: Something went wrong while executing your query. Please include `16C0:22C1:77BF52E:7B4322F:65CF3854` when reporting this issue.
```

Diffs=
6059f744d update mono to keep details in commit and not pr (#6637)
a394393a0 update mono scripts to be able to create "fixing" pr (#6636)
2277c08fc Apply same changes as downstream (#6634)
faf6ea317 Option to build harfbuzz with renames to avoid static lib collision. (#6630)
c51bda03e Fix downstream windows tests. (#6632)

Co-authored-by: Luigi Rosso <luigi-rosso@users.noreply.github.com>
Co-authored-by: Maxwell Talbot <talbot.maxwell@gmail.com> diff=\n\ndiff --cc .github/workflows/tests.yml
index 76659125,e20119eb..00000000
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@@ -58,8 -58,9 +58,12 @@@ jobs
          run: |
            cd dev/test
            C:\premake5.exe --scripts=..\..\build vs2022
++<<<<<<< ours
++=======
+           cd out/debug
++>>>>>>> theirs
            MSBuild rive.sln
-           build\bin\debug\tests.exe
+           tests.exe

    build-macos:
      runs-on: macOS-latest
2024-02-16 11:03:27 +00:00
Chris Dalton
3b3d835e54 Update the windows workflow to provide --scripts to premake 2022-11-18 10:26:43 -07:00
luigi-rosso
8f690ee101 Line breaker
Updating the line breaker to support:
- forced line breaks (return characters)
- store break points in context of runs (no searching when breaking)
- perform alignment (center/left/right)

In order to implement tests for this in Rive, I moved the Harfbuzz shaper into Rive (conditionally compiled out) but now we can test line-breaking logic in Rive core.

Since I had to update some of the build scripts, I ran the same new premake formatter we've been using.

<img width="910" alt="CleanShot 2022-09-19 at 21 03 53@2x" src="https://user-images.githubusercontent.com/454182/191164998-6b7b131b-b624-41c7-81d2-c2e01a050680.png">

Diffs=
a1d921c85 Line breaker (#4249)
2022-09-23 23:10:03 +00:00
luigi-rosso
5506f4dd56 Run tests for rive_tess
Adds ability to run tests when building rive_tess:

```
cd packages/runtime/tess/build/macosx
./build_tess.sh test
```

Also runs them in GitHub Actions.

Diffs=
265e9c7aa Run tests for rive_tess
2022-07-19 04:37:30 +00:00
Luigi Rosso
56a2cc55bb Removing some unnecessary commits from this PR.
Adding newline to fix diff.

Update tests.yml

Update tests.yml

See rive.make

Attempt using a pre-generate makefile.

Update tests.yml

Remove windows make

One more windows test.

Make with verbose.

Trying verbose ar.exe

Retry

Make without parallel

More debug

Trying to fix this wsl issue

Seeing which version of make we have on github actions

Use an instance of the artboard for SMI test

Attempt wsl

Attempt wsl

install unzip

Again..

try to get unzip

add to path instead of github_path with wsl

just move premake to bin

use correct filename for premake

use crlf

Back to crlf

space

Attempt using just powershell

premake with powershell

call msbuild

add msbuild to path

Proper msbuild config

more args

Build rive tests

use right solution name

Use any cpu for tests

remove the config and platform

Update tests for windows

LS right dir

look for exe

Tests!

Add “b” so windows reads in binary mode

More “rb” fixes

Final “rb” fix!
2022-04-25 12:07:55 -04:00
Luigi Rosso
6aa5e6ac65 Undoing change from windows build testing. 2022-04-05 08:57:03 -07:00
Luigi Rosso
f90cb3d654 Update tests.yml 2022-04-04 16:21:51 -07:00
Luigi Rosso
0951918ca8 Update tests.yml 2022-04-04 16:21:02 -07:00
Luigi Rosso
d47945b15b Update tests.yml 2022-04-04 16:20:20 -07:00
JCash
cb4f14bad7 removed cmake remnant 2021-08-31 16:14:33 -07:00
JCash
7ba5551fe4 Disabled Win32 tests since they currently don't build due to missing M_PI define 2021-08-31 16:14:33 -07:00
JCash
4c57316a49 Removed iOS 2021-08-31 16:14:33 -07:00
JCash
97f01a3df3 fix 2021-08-31 16:14:33 -07:00
JCash
56c1b3699d fix 2021-08-31 16:14:33 -07:00
JCash
f914bc5c5a fix 2021-08-31 16:14:33 -07:00
JCash
447bf29cf6 fix 2021-08-31 16:14:33 -07:00
JCash
249bba88a2 fix 2021-08-31 16:14:33 -07:00
JCash
51669e2120 fix 2021-08-31 16:14:33 -07:00
JCash
dd6a1de0cf Install premake5 using powershell 2021-08-31 16:14:33 -07:00
JCash
bdb80859ee Try download premake with powershell on windows 2021-08-31 16:14:33 -07:00
JCash
b9da781d69 remove sudo on windows 2021-08-31 16:14:33 -07:00
JCash
d45d536b1b Use unzip on windows 2021-08-31 16:14:33 -07:00
JCash
af88d0965f choco wget 2021-08-31 16:14:33 -07:00
JCash
59165f4c6f Use bash shell on windows 2021-08-31 16:14:33 -07:00
JCash
f15f4a1235 Use choco to install wget 2021-08-31 16:14:33 -07:00
JCash
936de2f083 Use wget action on windows. Skip WSL 2021-08-31 16:14:33 -07:00
JCash
175d8881b4 Use WSL on windows 2021-08-31 16:14:33 -07:00
JCash
ed32345558 removed apt-get steps as the tools should already be included 2021-08-31 16:14:33 -07:00
JCash
43c8708944 fix 2021-08-31 16:14:33 -07:00
JCash
ed1cf19499 fix 2021-08-31 16:14:33 -07:00
JCash
bfa7702162 fix 2021-08-31 16:14:33 -07:00
JCash
cc0f8c9150 fix 2021-08-31 16:14:33 -07:00
JCash
6ffffc2eaf Added install+build step. Added Win32/macOS platforms 2021-08-31 16:14:33 -07:00
Luigi Rosso
e9eb1853db Update tests.yml 2021-05-27 16:52:41 -07:00
Luigi Rosso
e758464500 Update tests.yml 2021-05-27 16:48:56 -07:00
Luigi Rosso
1cb3327f5c Update tests.yml 2021-05-27 16:46:50 -07:00
Luigi Rosso
d5027944b0 Create tests.yml 2021-05-27 16:45:43 -07:00