Skip to content

Commit 2c9d81f

Browse files
Bugfix/50 http and fin capability do not implement gettype string (#51)
1 parent c407a2c commit 2c9d81f

File tree

5 files changed

+14
-7
lines changed

5 files changed

+14
-7
lines changed

internal/capability/fin/fin.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,12 +24,16 @@ func New(finProtocol protocol.IFinProtocol) *FinCapability {
2424
return &FinCapability{finProtocol: finProtocol}
2525
}
2626

27+
func (FinCapability *FinCapability) GetType() string {
28+
return "soarca-fin"
29+
}
30+
2731
func (finCapability *FinCapability) Execute(
2832
metadata execution.Metadata,
2933
command cacao.Command,
3034
authentication cacao.AuthenticationInformation,
3135
target cacao.AgentTarget,
32-
variables map[string]cacao.Variable) (map[string]cacao.Variable, error) {
36+
variables cacao.Variables) (cacao.Variables, error) {
3337

3438
finCommand := finModel.NewCommand()
3539
finCommand.CommandSubstructure.Command = command.Command

internal/capability/http/http.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,12 @@ func init() {
3030
log = logger.Logger(component, logger.Info, "", logger.Json)
3131
}
3232

33+
func (httpCapability *HttpCapability) GetType() string {
34+
return "soarca-http-api"
35+
}
36+
3337
// What to do if there is no agent or target?
3438
// And maybe no auth info either?
35-
3639
func (httpCapability *HttpCapability) Execute(
3740
metadata execution.Metadata,
3841
command cacao.Command,

internal/fin/protocol/protocol.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ func init() {
3838
}
3939

4040
type IFinProtocol interface {
41-
SendCommand(fin.Command) (map[string]cacao.Variable, error)
41+
SendCommand(fin.Command) (cacao.Variables, error)
4242
}
4343

4444
type FinProtocol struct {
@@ -55,7 +55,7 @@ func New(guid guid.IGuid, topic Topic, broker Broker, port int) FinProtocol {
5555
return prot
5656
}
5757

58-
func (protocol *FinProtocol) SendCommand(command fin.Command) (map[string]cacao.Variable, error) {
58+
func (protocol *FinProtocol) SendCommand(command fin.Command) (cacao.Variables, error) {
5959

6060
client, err := protocol.Connect(command.CommandSubstructure.Authentication)
6161
if err != nil {

test/unittest/capability/fin/fin_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ func TestFinExecution(t *testing.T) {
4949

5050
//expectedCommand.CommandSubstructure.Context.GeneratedOn = ""
5151

52-
expectedVariableMap := map[string]cacao.Variable{"expected_variable": variable1}
52+
expectedVariableMap := cacao.NewVariables(variable1)
5353

5454
//mockGuid.On("New").Return(id)
5555
mockFinProtocol.On("SendCommand", expectedCommand).Return(expectedVariableMap, nil)

test/unittest/mocks/mock_finprotocol/mock_finprotocol.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ type MockFinProtocol struct {
1111
mock.Mock
1212
}
1313

14-
func (finProtocol *MockFinProtocol) SendCommand(command fin.Command) (map[string]cacao.Variable, error) {
14+
func (finProtocol *MockFinProtocol) SendCommand(command fin.Command) (cacao.Variables, error) {
1515
args := finProtocol.Called(command)
16-
return args.Get(0).(map[string]cacao.Variable), args.Error(1)
16+
return args.Get(0).(cacao.Variables), args.Error(1)
1717
}

0 commit comments

Comments
 (0)