FilesExpand file tree

main

/

merge-request-creation.test.ts

Copy path

Blame

More file actions

Blame

More file actions

Latest commit

 

History

History

History

296 lines (273 loc) · 8.26 KB

main

/

merge-request-creation.test.ts

Copy path

Top

File metadata and controls

• Code
• Blame

296 lines (273 loc) · 8.26 KB

Raw

Copy raw file

Download raw file

Open symbols panel

Edit and raw actions

1

2

3

4

5

6

7

8

9

10

11

12

13

14

15

16

17

18

19

20

21

22

23

24

25

26

27

28

29

30

31

32

33

34

35

36

37

38

39

40

41

42

43

44

45

46

47

48

49

50

51

52

53

54

55

56

57

58

59

60

61

62

63

64

65

66

67

68

69

70

71

72

73

74

75

76

77

78

79

80

81

82

83

84

85

86

87

88

89

90

91

92

93

94

95

96

97

98

99

100

101

102

103

104

105

106

107

108

109

110

111

112

113

114

115

116

117

118

119

120

121

122

123

124

125

126

127

128

129

130

131

132

133

134

135

136

137

138

139

140

141

142

143

144

145

146

147

148

149

150

151

152

153

154

155

156

157

158

159

160

161

162

163

164

165

166

167

168

169

170

171

172

173

174

175

176

177

178

179

180

181

182

183

184

185

186

187

188

189

190

191

192

193

194

195

196

197

198

199

200

201

202

203

204

205

206

207

208

209

210

211

212

213

214

215

216

217

218

219

220

221

222

223

224

225

226

227

228

229

230

231

232

233

234

235

236

237

238

239

240

241

242

243

244

245

246

247

248

249

250

251

252

253

254

255

256

257

258

259

260

261

262

263

264

265

266

267

268

269

270

271

272

273

274

275

276

277

278

279

280

281

282

283

284

285

286

287

288

289

290

291

292

293

294

295

296

import { beforeEach, describe, expect, it, vi } from 'vitest'

const {

getProjectSlugMock,

glabExecFileAsyncMock,

glabHostnameArgsMock,

glabHostEnvOptionsMock,

glabRepoExecOptionsMock,

acquireMock,

releaseMock

} = vi.hoisted(() => ({

getProjectSlugMock: vi.fn(),

glabExecFileAsyncMock: vi.fn(),

glabHostnameArgsMock: vi.fn((projectRef: { host: string }) => ['--hostname', projectRef.host]),

glabHostEnvOptionsMock: vi.fn((projectRef: { host: string }, connectionId?: string | null) =>

connectionId ? { env: { GITLAB_HOST: projectRef.host } } : {}

),

glabRepoExecOptionsMock: vi.fn((repoPath: string, connectionId?: string | null) =>

connectionId ? {} : { cwd: repoPath }

),

acquireMock: vi.fn(),

releaseMock: vi.fn()

}))

vi.mock('./client', () => ({

getProjectSlug: getProjectSlugMock

}))

vi.mock('./gl-utils', () => ({

acquire: acquireMock,

release: releaseMock,

glabExecFileAsync: glabExecFileAsyncMock,

glabHostnameArgs: glabHostnameArgsMock,

glabHostEnvOptions: glabHostEnvOptionsMock,

glabRepoExecOptions: glabRepoExecOptionsMock

}))

import { createGitLabMergeRequest } from './merge-request-creation'

describe('createGitLabMergeRequest', () => {

beforeEach(() => {

getProjectSlugMock.mockReset()

glabExecFileAsyncMock.mockReset()

glabHostnameArgsMock.mockClear()

glabHostEnvOptionsMock.mockClear()

glabRepoExecOptionsMock.mockClear()

acquireMock.mockReset()

releaseMock.mockReset()

acquireMock.mockResolvedValue(undefined)

getProjectSlugMock.mockResolvedValue({ host: 'gitlab.com', path: 'acme/widgets' })

})

it('creates a GitLab merge request with normalized refs', async () => {

glabExecFileAsyncMock.mockResolvedValueOnce({

stdout: 'https://gitlab.com/acme/widgets/-/merge_requests/42\n',

stderr: ''

})

await expect(

createGitLabMergeRequest(

'/repo-root',

{

provider: 'gitlab',

base: 'origin/main',

head: 'refs/heads/feature/create-mr',

title: ' Create MR UI ',

body: 'Body text',

draft: true

},

'local'

)

).resolves.toEqual({

ok: true,

number: 42,

url: 'https://gitlab.com/acme/widgets/-/merge_requests/42'

})

const [args, options] = glabExecFileAsyncMock.mock.calls[0]

expect(args).toEqual(

expect.arrayContaining([

'mr',

'create',

'-R',

'acme/widgets',

'--target-branch',

'main',

'--source-branch',

'feature/create-mr',

'--title',

'Create MR UI',

'--description',

'Body text',

'--draft'

])

)

expect(args).not.toContain('--hostname')

expect(options).toMatchObject({

cwd: '/repo-root',

timeout: 60_000,

idempotent: false

})

expect(options.env).toBeUndefined()

expect(acquireMock).toHaveBeenCalledOnce()

expect(releaseMock).toHaveBeenCalledOnce()

})

it('runs local WSL project merge request creation through the selected distro', async () => {

glabExecFileAsyncMock.mockResolvedValueOnce({

stdout: 'https://gitlab.com/acme/widgets/-/merge_requests/43\n',

stderr: ''

})

await expect(

createGitLabMergeRequest(

'/repo-root',

{

provider: 'gitlab',

base: 'main',

head: 'feature/wsl-create-mr',

title: 'WSL Create MR'

},

'local',

{ localGitExecOptions: { wslDistro: 'Ubuntu' } }

)

).resolves.toEqual({

ok: true,

number: 43,

url: 'https://gitlab.com/acme/widgets/-/merge_requests/43'

})

const [, options] = glabExecFileAsyncMock.mock.calls[0]

expect(options).toMatchObject({

cwd: '/repo-root',

wslDistro: 'Ubuntu',

timeout: 60_000,

idempotent: false

})

})

it('creates SSH-backed merge requests without using the remote path as a local cwd', async () => {

glabExecFileAsyncMock.mockResolvedValueOnce({

stdout: JSON.stringify({

iid: 45,

web_url: 'https://gitlab.com/acme/widgets/-/merge_requests/45'

}),

stderr: ''

})

await expect(

createGitLabMergeRequest(

'/remote/repo-root',

{

provider: 'gitlab',

base: 'main',

head: 'feature/ssh-create-mr',

title: 'SSH Create MR'

},

'ssh:ssh-1'

)

).resolves.toEqual({

ok: true,

number: 45,

url: 'https://gitlab.com/acme/widgets/-/merge_requests/45'

})

expect(getProjectSlugMock).toHaveBeenCalledWith('/remote/repo-root', 'ssh-1')

const [args, options] = glabExecFileAsyncMock.mock.calls[0]

expect(args).toEqual(

expect.arrayContaining([

'mr',

'create',

'-R',

'acme/widgets',

'--target-branch',

'main',

'--source-branch',

'feature/ssh-create-mr'

])

)

expect(options).toMatchObject({

timeout: 60_000,

idempotent: false

})

expect(options).not.toHaveProperty('cwd')

})

it('returns an existing merge request when GitLab reports a duplicate', async () => {

glabExecFileAsyncMock

.mockRejectedValueOnce(new Error('merge request already exists'))

.mockResolvedValueOnce({

stdout: JSON.stringify([

{

iid: 77,

web_url: 'https://gitlab.com/acme/widgets/-/merge_requests/77'

}

]),

stderr: ''

})

await expect(

createGitLabMergeRequest(

'/repo-root',

{

provider: 'gitlab',

base: 'main',

head: 'feature/existing',

title: 'Existing MR'

},

'local'

)

).resolves.toEqual({

ok: false,

code: 'already_exists',

error: 'A merge request already exists for this branch.',

reason: 'existing_review',

existingReview: {

number: 77,

url: 'https://gitlab.com/acme/widgets/-/merge_requests/77'

}

})

expect(glabExecFileAsyncMock.mock.calls[1][0]).toEqual(

expect.arrayContaining([

'mr',

'list',

'--source-branch',

'feature/existing',

'--target-branch',

'main'

])

)

})

// mr create rejects --hostname; self-hosted SSH workspaces need GITLAB_HOST.

it('creates a merge request through GITLAB_HOST on an SSH workspace', async () => {

getProjectSlugMock.mockResolvedValue({ host: 'gitlab.example.internal', path: 'acme/widgets' })

glabExecFileAsyncMock.mockResolvedValueOnce({

stdout: JSON.stringify({

iid: 51,

web_url: 'https://gitlab.example.internal/acme/widgets/-/merge_requests/51'

}),

stderr: ''

})

await expect(

createGitLabMergeRequest(

'/remote/repo-root',

{

provider: 'gitlab',

base: 'main',

head: 'feature/ssh-host',

title: 'SSH host MR'

},

'ssh:ssh-1'

)

).resolves.toMatchObject({ ok: true, number: 51 })

const [args, options] = glabExecFileAsyncMock.mock.calls[0]

expect(args.slice(0, 2)).toEqual(['mr', 'create'])

expect(args).not.toContain('--hostname')

expect(options.env?.GITLAB_HOST).toBe('gitlab.example.internal')

})

// Duplicate lookup uses mr list, which also requires GITLAB_HOST instead of --hostname.

it('looks up an existing merge request through GITLAB_HOST on an SSH workspace', async () => {

glabExecFileAsyncMock

.mockRejectedValueOnce(new Error('merge request already exists'))

.mockResolvedValueOnce({

stdout: JSON.stringify([

{

iid: 77,

web_url: 'https://gitlab.example.internal/acme/widgets/-/merge_requests/77'

}

]),

stderr: ''

})

getProjectSlugMock.mockResolvedValue({ host: 'gitlab.example.internal', path: 'acme/widgets' })

await expect(

createGitLabMergeRequest(

'/repo-root',

{

provider: 'gitlab',

base: 'main',

head: 'feature/existing',

title: 'Existing MR'

},

'ssh:ssh-1'

)

).resolves.toMatchObject({ ok: false, code: 'already_exists', reason: 'existing_review' })

const [args, options] = glabExecFileAsyncMock.mock.calls[1]

expect(args).not.toContain('--hostname')

expect(options.env?.GITLAB_HOST).toBe('gitlab.example.internal')

})

})