Skip to content

Commit

Permalink
chore: need includeDevDependencies:true to use vat-data
Browse files Browse the repository at this point in the history
Swingset has `config.includeDevDependencies` to instruct bundle-source
to look for `devDependencies`, not just `dependencies`, and now we
need it in any unit test that uses the vat-data package.

We'll need it in other tests as well, when their vats import code from
outside of the swingset package.
  • Loading branch information
warner committed Apr 12, 2022
1 parent 81cee9c commit 216d8e1
Show file tree
Hide file tree
Showing 6 changed files with 9 additions and 0 deletions.
2 changes: 2 additions & 0 deletions packages/SwingSet/test/upgrade/test-upgrade.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ function get(capdata, propname) {

async function testUpgrade(t, defaultManagerType) {
const config = {
includeDevDependencies: true, // for vat-data
defaultManagerType,
bootstrap: 'bootstrap',
defaultReapInterval: 'never',
Expand Down Expand Up @@ -93,6 +94,7 @@ test('vat upgrade - xsnap', async t => {

test('failed upgrade - lost kind', async t => {
const config = {
includeDevDependencies: true, // for vat-data
defaultManagerType: 'xs-worker',
bootstrap: 'bootstrap',
defaultReapInterval: 'never',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ function getImportSensorKref(impcapdata, i) {

test('collection entry slots trigger doMoreGC', async t => {
const config = {
includeDevDependencies: true, // for vat-data
bootstrap: 'bootstrap',
// defaultReapInterval: 'never',
// defaultReapInterval: 1,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ function getImportSensorKref(impcapdata, i) {

test('VO property deletion is not short-circuited', async t => {
const config = {
includeDevDependencies: true, // for vat-data
bootstrap: 'bootstrap',
// defaultReapInterval: 'never',
// defaultReapInterval: 1,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ function bfile(name) {

async function testUpgrade(t, defaultManagerType) {
const config = {
includeDevDependencies: true, // for vat-data
defaultManagerType,
bootstrap: 'bootstrap',
vats: {
Expand Down
3 changes: 3 additions & 0 deletions packages/SwingSet/test/virtualObjects/test-representatives.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ function slot0(iface, kid) {

test.serial('exercise cache', async t => {
const config = {
includeDevDependencies: true, // for vat-data
bootstrap: 'bootstrap',
vats: {
bootstrap: {
Expand Down Expand Up @@ -350,6 +351,7 @@ test('virtual object gc', async t => {
*/

const config = {
includeDevDependencies: true, // for vat-data
bootstrap: 'bootstrap',
defaultManagerType: 'xs-worker',
vats: {
Expand Down Expand Up @@ -403,6 +405,7 @@ test('virtual object gc', async t => {
// Check that facets which don't reference their state still kill their cohort alive
async function orphanTest(t, mode) {
const config = {
includeDevDependencies: true, // for vat-data
bootstrap: 'bootstrap',
vats: {
bob: {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ function capargs(args, slots = []) {

test('weakMap in vat', async t => {
const config = {
includeDevDependencies: true, // for vat-data
bootstrap: 'bootstrap',
vats: {
bootstrap: {
Expand Down

0 comments on commit 216d8e1

Please sign in to comment.