Skip to content

Conversation

@ahyalfan
Copy link
Member

Deskripsi (Description)

Pull request ini mencakup beberapa perubahan penting terkait struktur dan kualitas kode testing:

  • Memindahkan seluruh file unit test ke dalam folder tests/ untuk meningkatkan keteraturan dan memudahkan pengelolaan testing.
  • Menyesuaikan Makefile dan konfigurasi testing agar mendukung struktur folder baru.
  • Memperbarui import pada beberapa file agar sesuai dengan lokasi baru dari package dan test.
  • Mengubah loop benchmark dari pola lama for i := 0; i < b.N; i++ ke penggunaan b.Loop(...) yang merupakan fitur idiomatik baru di Go 1.22, sehingga kode benchmark menjadi lebih ringkas dan modern.

Perubahan ini bertujuan untuk meningkatkan maintainability proyek tanpa mengubah perilaku fungsional dan hasil pengujian.

Checklist:

Umum:
  • Saya menambah algoritma terbaru.
  • Saya memperbaiki algoritma yang sudah ada.
  • Saya memperbaiki dokumentasi.
  • Saya menambah dokumentasi.
Contributor Requirements (Syarat Kontributor) dan Lain-Lain:
  • Saya sudah membaca (I have read) CONTRIBUTING dan sudah menyetujui semua syarat.
  • Saya telah menambahkan docstring yang memberikan penjelasan maksud dari kode yang saya buat.
  • Saya menggunakan bahasa Indonesia untuk memberikan penjelasan dari kode yang saya buat.

Environment

Saya menggunakan (I'm using):

  • os = linux
  • Go version = go version 1.24.3

linked issue #NOMOR_ISSUE

ahyalfan added 4 commits July 12, 2025 02:09
Perubahan ini memindahkan semua file test yang sebelumnya tersebar ke
dalam satu folder /test
untuk merapikan struktur proyek dan mempermudah pengelolaan.

Tidak ada perubahan pada logika pengujian — hanya lokasi file dan jalur
impor (jika perlu disesuaikan).

Tujuan refactor ini:
- Mengorganisir file test agar tidak bercampur dengan kode produksi
- Mempermudah konfigurasi CI dan penemuan test otomatis
- Mempersiapkan struktur proyek agar lebih scalable di masa depan

Catatan: Pastikan skrip atau konfigurasi yang bergantung pada lokasi
test sebelumnya juga diperbarui.
menambahkan beberapa fungsi untuk mempermudah unit test, misalnya
assert,ok,equals
Memperbarui fungsi benchmark dengan menggunakan b.Loop yang
diperkenalkan di Go 1.22 sebagai pengganti idiom for i := 0; i < b.N;
i++.

Perubahan ini membuat kode lebih idiomatik, lebih mudah dibaca, dan
mendukung fitur benchmark yang lebih modern.

Tidak ada perubahan perilaku pada hasil benchmark.
@ahyalfan ahyalfan requested a review from nuraskiah as a code owner July 11, 2025 19:44
@mergify mergify bot added the Golang files label Jul 11, 2025
@ahyalfan ahyalfan added the Ok to merge ok to merge to branch main label Jul 11, 2025
@ahyalfan ahyalfan self-assigned this Jul 12, 2025
@slowy07 slowy07 added the ready-to-merge siap merge label Jul 12, 2025
Copy link
Member

@slowy07 slowy07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mergify mergify bot merged commit 160dee6 into main Jul 12, 2025
4 checks passed
@mergify
Copy link

mergify bot commented Jul 12, 2025

terima kasih atas kontribusinya @ahyalfan!

@slowy07 slowy07 deleted the tests branch July 12, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants