You can remove the singe dot, it's not needed.

This can be changed to %make test to simplify it a bit.

The pkgconfig builddeps should be ordered alphabetically above fpc.

As Josh pointed out in D6751 to me, the sorting must be case-sensitive (A-Z, then a-z).

It's not needed for %cmake_ninja, but needed for regular %cmake. Without the dot you'll get warning:

CMake Warning:
No source or binary directory provided. Both will be assumed to be the
same as the current working directory, but note that this warning will
become a fatal error in future CMake releases.

That's because our regular cmake macro doesn't specify build directory and builds everything in-source by default. This is not the case for cmake_ninja because it builds in solusBuildDir directory and specifies this in the macro expansion (source).
While for now writing %cmake without specifying directory isn't critical, according to warning it can cause problems with future CMake releases.

This is usually used when autotests can fail in normal situations for some reason, so this ensures that test error won't be fatal for entire build process. I don't know why this was added here, so I prefer to leave it as it is.