ef7b89cc03
this changes globalFlags to be a ulext.External interface value that is passed to each command. rather than have the ulext.External have a Setup call in the way that the projectProvider used to we make all of the state arguments to the functions and have the commands call setup themselves. the reason it is in its own package is so that cmd/uplinkng can import cmd/uplinkng/ultest but cmd/uplinkng/ultest needs to refer to whatever the interface type is to call the function that creates the commands. there's also quite a bit of shuffling around of code and names. sorry if that makes it tricky to review. there should be no logic changes, though. a side benefit is there's no longer a need to do a type assertion in ultest to make it set the fake filesystem to use. that can be passed in directly now. additionally, this makes the access commands much easier to test. Change-Id: I29cf6a2144248a58b7a605a7ae0a5ada5cfd57b6
41 lines
771 B
Go
41 lines
771 B
Go
// Copyright (C) 2021 Storj Labs, Inc.
|
|
// See LICENSE for copying information.
|
|
|
|
package main
|
|
|
|
import (
|
|
"github.com/zeebo/clingy"
|
|
"github.com/zeebo/errs"
|
|
|
|
"storj.io/storj/cmd/uplinkng/ulext"
|
|
)
|
|
|
|
type cmdMb struct {
|
|
ex ulext.External
|
|
|
|
access string
|
|
|
|
name string
|
|
}
|
|
|
|
func newCmdMb(ex ulext.External) *cmdMb {
|
|
return &cmdMb{ex: ex}
|
|
}
|
|
|
|
func (c *cmdMb) Setup(params clingy.Parameters) {
|
|
c.access = params.Flag("access", "Which access to use", "").(string)
|
|
|
|
c.name = params.Arg("name", "Bucket name (sj://BUCKET)").(string)
|
|
}
|
|
|
|
func (c *cmdMb) Execute(ctx clingy.Context) error {
|
|
project, err := c.ex.OpenProject(ctx, c.access)
|
|
if err != nil {
|
|
return errs.Wrap(err)
|
|
}
|
|
defer func() { _ = project.Close() }()
|
|
|
|
_, err = project.CreateBucket(ctx, c.name)
|
|
return err
|
|
}
|