Test files should not be pulled into builds

[mduvergey]

Hi,

Installing NSubstitute with Unity package manager is required in order for Fluid Behavior Tree to compile properly.

Installation guidelines should mention that.

Best regards

[ashblue]

This should only be required if you're trying to run the test suite (at that point I highly suggest cloning down this repo instead) or you manually pulled the files into your project. How did you install the package? Hopefully you used the guide here?

https://github.com/ashblue/fluid-behavior-tree#installing

[mduvergey]

I installed the package following guide instructions, adding scopedRegistries entry in file manifest.json then using Unity Package Manager to get the package.

[ashblue]

Weird, haven't seen this before. Wondering if you somehow included the test files with your asmdef files. I'll try to reproduce on my end this weekend. What's your Unity version?

[mduvergey]

It's 2018.4.22f1.

[ashblue]

Uh oh, I think this package is marked as 2019 required at least. Big changes to the package manager since then. I'll update the install docs if this is the case.

[ashblue]

Okay min is 2018.1, will investigate and update more here this weekend https://github.com/ashblue/fluid-behavior-tree/blob/master/Assets/FluidBehaviorTree/package.json#L6

[ashblue]

A fresh install of 2019.3.15f1 with the library works fine. Proceeding to check 2018.4.22f1 to see if I can reproduce.

[ashblue]

@mduvergey I tried your specific version and I can't reproduce this issue. Can you post the specific error message? I think it may be something specific to your setup. Are you using .asmdef files? If so don't include any of the test ones.